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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

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:

100px
Media 100px

180px
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

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
FileSize
80.97 KB

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

idflood’s picture

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.