During the display of the media browser, we're using the square_thumbnail image style, which we've specified to scale to 180x180. However, the css in the browser further shrinks it to a 100px width. Which do we want?

The 100px?

Media browser 100px

or the 180px?

Media browser 180px

I'll attach a patch for either, once we have a consensus: we either need to change the image style to 100px, or change the css to 180px.

Files: 
CommentFileSizeAuthor
#7 media_zoom_slider-1024844-7.patch3.01 KBidflood
PASSED: [[SimpleTest]]: [MySQL] 0 pass(es).
[ View ]
#7 zoom-icons.png224 bytesidflood
#6 media-zoom-mockup1.jpg80.97 KBidflood
#5 media_zoom_slider-1024844-5.patch2.23 KBidflood
PASSED: [[SimpleTest]]: [MySQL] 0 pass(es).
[ View ]

Comments

Alternatively, we could keep both, and add a style switcher in the browser, which could be cool, and is similar to a demo by mverbaar during d7ux. Now that I think about it, I'm leaning in that direction...

images here:

100px
Media 100px

180px
Media 180px

It would be great if we could use a jQuery UI slider to adjust the size from say 40 or 60 to the full 180. Thoughts?

Version:7.x-1.x-dev» 7.x-2.x-dev
Category:bug» feature

StatusFileSize
new2.23 KB
PASSED: [[SimpleTest]]: [MySQL] 0 pass(es).
[ View ]

Here is a first patch that adds a slider to control the zoom level for the thumbs (from 50px to 180px by increment of 10px).

I think there will be something to do to reapply the style to dynamically loaded thumb but I may be wrong. And this will also obviously need some specific styling.

Status:Active» Needs review
StatusFileSize
new80.97 KB

maybe it can looks something like that. what do you think?

StatusFileSize
new224 bytes
new3.01 KB
PASSED: [[SimpleTest]]: [MySQL] 0 pass(es).
[ View ]

here is a patch that make look the slider like in the proposed mockup above. The image needs to be added in the "images" folder.

Assigned:Unassigned» Dave Reid

Assigning for review.

Applied and tested #7 "media_zoom_slider-1024844-7.patch" against the current 7.x-1.x...
Works as described in #6.

Is this still happening/relevant?

Status:Needs review» Needs work

well it doesnt not apply at all

Issue summary:View changes
Status:Needs work» Fixed

This was fixed as part of #1881152: Browser display cleanup.

Status:Fixed» Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.