mesaglam / mad-components

Automatically exported from code.google.com/p/mad-components
0 stars 0 forks source link

Image sizing doesn't work in the <image height="20"> tage #23

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
set the size of a GIF to something other than its native size

What is the expected output? What do you see instead?
scale should adjust but it's 1 instead

What version of the product are you using? On what operating system?
0.7.1

Please provide any additional information below.
Patch attached but it looks to me like Attributes.widthH and Attributes.heightV 
should be used everywhere because otherwise set values for width and height 
will be ignored, right?

Also, the way height and width are handled in Attributes makes it possible for 
_height to have a value of 50 and height to have a value of 300. That seems 
confusing to me and may deserve another look.

thanks!

Original issue reported on code.google.com by siredl...@gmail.com on 6 Mar 2012 at 5:08

Attachments:

GoogleCodeExporter commented 9 years ago
Update: I'm not sure that's a bug anymore. With the non-patched code, this 
works: <imageLoader id="image_url">50,50</imageLoader>. But then what are the 
attribute height and width for, why would they behave differently? Thanks,  - S.

Original comment by siredl...@gmail.com on 6 Mar 2012 at 5:26

GoogleCodeExporter commented 9 years ago

Original comment by doc.andr...@gmail.com on 12 Jul 2012 at 11:29