Page MenuHomePhabricator

Image display in lightbox is smaller than on on file: page
Closed, ResolvedPublic

Description

view of photo in lightbox

I think part of the idea of using a popup lightbox instead of sending people to File: description pages is that we have the entire window available to show an image, with a lot less UI chrome around it.

However, I find that often the view in the lightbox is smaller than the preview I see on the file: page, and there's a lot of whitespace around the edges of the box.

See attached images comparing view in lightbox to existing view on file page.


Version: unspecified
Severity: normal
URL: https://test.wikipedia.org/wiki/Lightbox_demo

Attached:

Screen_Shot_2013-11-01_at_4.15.44_PM.png (1×1 px, 829 KB)

Details

Reference
bz56499

Event Timeline

bzimport raised the priority of this task from to High.Nov 22 2014, 2:31 AM
bzimport added a project: MediaViewer.
bzimport set Reference to bz56499.
bzimport added a subscriber: Unknown Object (MLST).

Created attachment 13666
view of same photo on file page

Attached:

Screen_Shot_2013-11-01_at_4.16.03_PM.png (1×1 px, 827 KB)

Agreed. Depending on the aspect ratio of the image, this can also be true even comparing the thumb size in an _article_ vs. the size displayed in the image viewer. See:

https://test.wikipedia.org/wiki/Lightbox_demo#Different_aspect_ratios

Thanks, Brion and Erik. You make some excellent points, and your suggestions are well taken. We will work with our design team to make the images more prominent, and make the meta-data less prominent (e.g. moving it down below the image, where the user can scroll down if they want to read it).

where the user can scroll down if they want to read it

This would mean changing the structure of the lightbox interface again, but all right :)

I'd much prefer not messing with that, though - maybe pop up the metadata on a hover state, or with a click event, instead? i.e. put the metadata div down below, and pop it up over the image a little bit when it gets moused over?

Change 95081 had a related patch set uploaded by MarkTraceur:
Reorganize UI to be...way sexier.

https://gerrit.wikimedia.org/r/95081

Linked patch should do the trick, and generally makes everything better. Basically will cure your leprosy if you want it to. My gods.

See the latest on http://multimedia-alpha.wmflabs.org

Change 95081 merged by jenkins-bot:
Reorganize UI to be...way sexier.

https://gerrit.wikimedia.org/r/95081

Patch was merged a while ago - is there more work left to do here (if yes: please reset the bug report status to NEW or ASSIGNED), or can you close this ticket as RESOLVED FIXED?

Gilles raised the priority of this task from High to Unbreak Now!.Dec 4 2014, 10:11 AM
Gilles moved this task from Untriaged to Done on the Multimedia board.
Gilles lowered the priority of this task from Unbreak Now! to High.Dec 4 2014, 11:23 AM