owncloud
[+] Exif Description
show exif description (if exists) instead filename on preview and slideshow
TODO
Designers need to figure out how to properly show that description.
- Centred?
- 3 lines? In this example it would just be the first line over 3 narrower lines
- Scroller or rollover? Rollover would fill the screen with that description since we have to keep it narrow. Scroller requires 2 additional touch areas instead of an ugly scrollbar
A lot of galleries show the exif data in a side-bar.
Thanks! We'll first need to discuss if that's something we want to add, but in the meantime, I'll just write down notes about the changes needed before this can be merged.
@jancborchardt and his team can check the design.
Thanks for the fixes.
How long can descriptions be? I think you might need to use a different CSS style if we're talking about a paragraph.
3 seconds as control. enough to read. then goes out to show the full picture.
3 seconds as control. enough to read. then goes out to show the full picture.
Wait, so you mean, this is blocking the rendering of the picture? First the description is shown for 3 seconds, then it's replaced by the image and cannot be seen again?
no, no. :) description show over pictures, and may close the necessary detail in the picture... after hide its may be show by mouse movement.
OK, but how big can that description be? If it can be as long as it wants, we may need to limit it to maybe 2 lines and reveal the rest on hover.
Yes, but that's for the title, so it's not a problem if we're missing 10% of the filename because there isn't enough space, but if we only get one line out of 10 for a description, then it's going to be frustrating for users.
@demattin @setnes @patman15 Anybody with pictures with EXIF descriptions so that we can properly test this PR?
@0xb0ba can you post a screenshot of this change so it’s easier to review?
Thanks for that @0xb0ba.
Designers need to figure out how to properly show that description.
- Centred?
- 3 lines? In this example it would just be the first line over 3 narrower lines
- Scroller or rollover? Rollover would fill the screen with that description since we have to keep it narrow. Scroller requires 2 additional touch areas instead of an ugly scrollbar
A lot of galleries show the exif data in a side-bar. I think it would make it easier to show the meta-data we have.
@oparoz, I'm not a designer. I can not answer these questions. in most cases it is the description of a few words, scroller or rollover is likely not needed.
exif data need not always. its cool in popup side-bar. description should always, especially in the slideshow.
I’m not sure we should show it there to be honest. What’s the usecase for exif description like that, I have never seen it really. How do other photo apps approach this?
Thanks for the screenshot @0xb0ba. What happens on Picasa when you switch to portrait mode?
@oparoz Sorry, I only have loads of pictures with GPS tags but no comments ...
Thanks. That works quite well for landscape pictures shown in portrait mode, but would cover 1/3 of a portrait.
The default timer is for the whole interface, so it is a problem when zooming in/out as there is a large block of text masking part of the image.
its not problem too - https://github.com/0xb0ba/gallery/commit/add90f0939ea2a62052b6148218b17c11dd8a4ec