verginman09 / minimalist-facebook

Automatically exported from code.google.com/p/minimalist-facebook
1 stars 1 forks source link

Feature Request: Photo zoom layout options #47

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
I'd like an alternate Photo Zoom layout so that the caption on the popup 
doesn't cover up the top of the photo.

I tried messing with the css a bit but I couldn't get it how I wanted (fully 
visible photo, caption below the photo). It worked fine for short captions, but 
the layout broke as soon as the caption text was longer than the photo was 
wide. I'd need some javascript to lock the width of the caption DIV so it 
doesn't go wider than the photo.

Original issue reported on code.google.com by estma...@gmail.com on 12 Apr 2011 at 3:38

GoogleCodeExporter commented 9 years ago

Original comment by anst...@gmail.com on 12 Apr 2011 at 4:18

GoogleCodeExporter commented 9 years ago
yeah, the tooltip is blocking the view of photos when auto-zoom enabled..
Why don't when auto zoom is enabled, disable the tooltip.. vice-versa..

Original comment by m...@irmanahmad.com on 27 Apr 2011 at 3:36

GoogleCodeExporter commented 9 years ago
Agreed, the caption can get in the way sometimes. I'd just pop it under the 
image in some extra white space - then you can see the whole picture.

Original comment by touchmei...@gmail.com on 31 May 2011 at 6:28

GoogleCodeExporter commented 9 years ago
I have the same issue, the caption covers part of the picture, it should be 
moved to a whitebox area under the image.

Original comment by gjef...@gmail.com on 4 Aug 2011 at 1:43

Attachments:

GoogleCodeExporter commented 9 years ago
I added this to the "Custom CSS" box:

#zoom div {position:static;}

It's not ideal but it works fine for most photos. It gets a little screwy if 
the caption is really long or if the photo is really big (caption ends up going 
off the bottom of the screen).

Original comment by estma...@gmail.com on 6 Aug 2011 at 4:32

GoogleCodeExporter commented 9 years ago
I exaggerated a bit by making the browser window smaller, but this really 
illustrates why the caption shouldn't be an overlay. The extension should also 
strip out the title attribute so the browser doesn't show the default, yellow 
tooltip. Most of the image is covered up! :P

Original comment by estma...@gmail.com on 6 Aug 2011 at 4:43

Attachments: