It would be nice to let the user choose which image derivative to open in the thickbox: the original image is often too large

Comments

Egon Bianchet’s picture

Status: Active » Needs review
StatusFileSize
new2.03 KB

Not sure if this is the best approach, but it works.

kleingeist’s picture

Hi this seems to be fine on the first look.

BUt you should wrap the whole derivate options form block inside an
to prevent errors on installations where the image.module isnt installed

kleingeist’s picture

k inline < ?php tags wont work... good to know:
----
[...]înside an

if (module_exists('image')) { 
  $options = array();
  $sizes = _image_get_sizes
  ...
}

to prevent errors on installations where the image.module isnt installed

Egon Bianchet’s picture

StatusFileSize
new2.26 KB

Updated with check for image module and make it possible to choose the "original" size too

frjo’s picture

Committed to HEAD, please try it out and report back here.

Egon Bianchet’s picture

Status: Needs review » Fixed

It's working fine, thanks

Anonymous’s picture

Status: Fixed » Closed (fixed)