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.

#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
media-100.png219.72 KBaaron
media-180.png431.26 KBaaron
Members fund testing for the Drupal project. Drupal Association Learn more


aaron’s picture

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...

aaron’s picture

images here:

Media 100px

Media 180px

Jackinloadup’s picture

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?

JacobSingh’s picture

Version: 7.x-1.x-dev » 7.x-2.x-dev
Category: bug » feature
idflood’s picture

2.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.

idflood’s picture

Status: Active » Needs review
80.97 KB

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

idflood’s picture

224 bytes
3.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.

Dave Reid’s picture

Assigned: Unassigned » Dave Reid

Assigning for review.

jerryitt’s picture

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

dddave’s picture

Is this still happening/relevant?

ParisLiakos’s picture

Status: Needs review » Needs work

well it doesnt not apply at all

ezra-g’s picture

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.