-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Overhaul media details view pane #873
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
A couple of things I notice about the media details view - firstly, that we don't respect the portrait / landscape nature of the image. For example, this picture was taken landscape. Looks good when my phone is in landscape As I rotate my phone the details view does nothing to imply the original orientation, it just truncates the picture. Since this is the normal orientation for a phone, we truncate by default: Google Photos does the right thing - it shrinks to fit and letterboxes the image - making it clear what the original orientation was. Secondly, since we host the details view in a fragment, as part of the |
There are 3 operations available on the detail view right now and it sounds like we're talking about adding a 4th
Google Photos also offers 4 actions on a bottom navigation (share, edit, info, delete), and a bunch of others in an overflow menu. The detailed picture information is on its own full-screen view. The bottom navigation definitely cleans things up. Google Photos does have a nice feature where an image tagged with a location will show a map view on the (full-screen) image info screen. Thinking about navigation, the details screen is in a subsection of the navigation hierarchy and ought to show the <- "up" navigation arrow button not the hamburger (nav-drawer) button. Crazy idea - since the main image list already shows the title, why not expand the list item in-place to a card view with other details? A |
I was playing with the main Wikipedia app today - particularly around how it handles photos: The details are minimal (as you would expect - the focus isnt on pictures) but it does highlight the idea of letterboxing photos in landscape mode. They also do a nice job of presenting content in card views on the main screen too. I like how they did their bottom navigation view on the main screen too. |
New bug logged - #894 (Media details lost on device rotation) If we are overhauling the media details screen we might want to fix the bug, or at least make sure we dont cause a regression and re-introduce it. |
Featured pictures are meant to be enjoyed as much as the screen allows, and currently the media details activity is indeed far from optimal: |
I think this can be considered solved, even category edition seems to have found its place. |
The current media details view pane feels rather clunky, and would be difficult to implement new features such as #161 . I'm not sure exactly what sort of redesign I have in mind yet (perhaps someone with a better design eye or familiarity with other photography apps can chime in?), but preferably it should be able to pave the way for #161 .
The text was updated successfully, but these errors were encountered: