Okay, this may seem stupid BUT ...

Is it possible to specify 2 aspect ratios? EG 5:3 or 3:5 would be allowable.

Put another way, both Tall and Wide images are okay, BUT (as per my current requirement) ExtraTall or ExtraWide images are not okay.

CommentFileSizeAuthor
#2 relative_ratio_and_undefined_index_fix-1106924-2.patch15.8 KBzhgenti
#1 relative_ratio_and_undefined_index_fix-1106924-1.patch15.78 KBzhgenti

Comments

zhgenti’s picture

zhgenti commented
StatusFileSize
new relative_ratio_and_undefined_index_fix-1106924-1.patch15.78 KB

Here is a patch, which adds additional controls for relative ratio like 4:3 or 16:9. These controls are hidden by default with #states attribute, and appears only when "enforce ratio" is checked and output resolution isn't set. This patch contains fixes from this path http://drupal.org/node/1139762#comment-4398398.

zhgenti’s picture

zhgenti commented
StatusFileSize
new relative_ratio_and_undefined_index_fix-1106924-2.patch15.8 KB

Small fix

barwonhack’s picture

barwonhack commented

I'll try this out soon. Thanks :)

yhager’s picture

yhager commented
Status: Active » Needs work

Sorry about the delay in the review, I've been busy.
Can you please remove the white-space changes from the patch, and leave only the actual code changes?

yhager’s picture

yhager commented

Actually, I wish you had separated the bug fix from #1139762: Errors when output resolution is set to 0 and this feature request.

bryancasler’s picture

bryancasler commented

subscribe