778534 - Use image instead of html:img in instant messaging

Status: RESOLVED DUPLICATE of bug 956579

(Thunderbird :: Instant Messaging, defect)

Description

[Ian Neal]

Over in bug 774063 it was suggested that html:img could be removed from imbuddytooltip.xml

Comment 1

[Ian Neal]

Attachment: Switch to xul:image

Simple switch to using xul:image instead of html:img

Comment 2

[Mark Banner (:standard8)]

Comment on attachment 680792 [details] [diff] [review]
Switch to xul:image

Florian's better to review this code.

Comment 3

[Wayne Mery (:wsmwk)]

(In reply to Mark Banner (:standard8) from comment #2)
> Comment on attachment 680792 [details] [diff] [review]
> Switch to xul:image
> 
> Florian's better to review this code.

Comment 4

[Florian Quèze [:florian]]

Comment on attachment 680792 [details] [diff] [review]
Switch to xul:image

The HTML img tag keeps the aspect ratio while scaling the image down. The XUL image tag doesn't.

Sorry for the delay. I suspected this, but wanted to actually try the patch to verify that the behavior of the XUL image tag hasn't changed since when I initially wrote that code.

It seems that since bug 585069 got fixed there should be a better way to do the scaling, but my quick attempts to use <xul:image maxheight="48" maxwidth="48" instead of http://hg.mozilla.org/comm-central/annotate/ff643eff7154/mail/components/im/content/imbuddytooltip.xml#l140 weren't successful.

Comment 5

[[e-mail]]

(In reply to Florian Quèze from comment #4)
> It seems that since bug 585069 got fixed there should be a better way to do
> the scaling, but my quick attempts to use <xul:image maxheight="48"
> maxwidth="48" instead of
> http://hg.mozilla.org/comm-central/annotate/ff643eff7154/mail/components/im/
> content/imbuddytooltip.xml#l140 weren't successful.

Possibly because of the inline style on the parent?

Comment 6

[Ian Neal]

As the code this was patching was rewritten in bug 956579, marking as a dupe of that bug.