592601 - Add "face" to the list of white-listed attributes

Status: VERIFIED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[kevin zhang]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; zh-CN; rv:1.9.2.8) Gecko/20100722 Firefox/3.6.8
Build Identifier: 3.1.3

I think there is a problem in the beta version of Thunderbird 3.1.3
we use a HTML file as our signature, in the file, we define the fonts should be Calibri, but when we compose Email, the fonts in the signature was turned into another fonts, such as the system default fonts, but the body of the Email can use the user-defined fonts, like Calibri

Reproducible: Always

Actual Results:  
no change


no additional information

Comment 2

[Ludovic Hirlimann [:Usul]]

Joe is this one of our known issues ?

Comment 3

[Joe Sabash [:JoeS1]]

I'm unable to test extensively at the moment (I'm on an interim PC) But this looks like a new issue to me. The Font face tag seems to be removed from the original html file when applied as a sig.

kevin, You could help with this by comparing the message source of your html file with the html in the applied sig.(View>>Message source)

Comment 4

[Joe Sabash [:JoeS1]]

A little further testing on this in 
Mozilla/5.0 (Windows NT 5.0; rv:2.0b5pre) Gecko/20100831 Thunderbird/3.2a1pre ID:20100831032343

There have been some recent changes in Core Editor that most likely is involved in this bug (bug 580442) Certain html tags are now sanitized by the editor, and I guess we should add the "font face" tag to that bug.

A work-around would be to use inline styles in the sig instead of a font tag.
Here's an example that works for me.
<span style="font-family: Comic Sans MS;">JoeS
</span>

This preserves the font selection when used in a signature.
Marking as new although this will probably be determined to be a core bug.

Comment 5

[kevin zhang]

(In reply to comment #3)
> I'm unable to test extensively at the moment (I'm on an interim PC) But this
> looks like a new issue to me. The Font face tag seems to be removed from the
> original html file when applied as a sig.
> 
> kevin, You could help with this by comparing the message source of your html
> file with the html in the applied sig.(View>>Message source)

Hi Joe, yes, I just compared the html file with the html appied as a sig, the font tag was removed after we applied it as a sig in the mail, only left the font color tag
Thanks

Comment 6

[kevin zhang]

(In reply to comment #4)
> A little further testing on this in 
> Mozilla/5.0 (Windows NT 5.0; rv:2.0b5pre) Gecko/20100831 Thunderbird/3.2a1pre
> ID:20100831032343
> 
> There have been some recent changes in Core Editor that most likely is involved
> in this bug (bug 580442) Certain html tags are now sanitized by the editor, and
> I guess we should add the "font face" tag to that bug.
> 
> A work-around would be to use inline styles in the sig instead of a font tag.
> Here's an example that works for me.
> <span style="font-family: Comic Sans MS;">JoeS
> </span>
> 
> This preserves the font selection when used in a signature.
> Marking as new although this will probably be determined to be a core bug.

yes, the span tag working, thanks

Comment 7

[(no longer active)]

Attachment: Patch (v1)

Comment 8

[Johnny Stenback (:jst)]

Doesn't sound like a blocker to me. If it should be, please say why and renominate.

Comment 9

[(no longer active)]

(In reply to comment #8)
> Doesn't sound like a blocker to me. If it should be, please say why and
> renominate.

I think it should be a blocker, as it is a regression from bug 520189.  For example, if you copy some HTML including a <font face> tag and paste it into an editable field, the face attribute will be lost.

Comment 10

[(no longer active)]

Attachment: For check-in

Comment 11

[Johnny Stenback (:jst)]

Ok, thanks for the clarification! Blocking.

Comment 12

[(no longer active)]

http://hg.mozilla.org/mozilla-central/rev/c63f70603dd4

Comment 13

[Daniel Veditz [:dveditz]]

Comment on attachment 472682 [details] [diff] [review]
For check-in

Approved for 1.9.2.10 and 1.9.1.13, a=dveditz for release-drivers

Comment 14

[(no longer active)]

http://hg.mozilla.org/releases/mozilla-1.9.2/rev/9d4aa3b682a8
http://hg.mozilla.org/releases/mozilla-1.9.1/rev/d978f2a4078c

Comment 16

[Robert Kaiser]

I also re-imported this on the 1.9.1.12 relbranch as http://hg.mozilla.org/releases/mozilla-1.9.1/rev/1389736dbf5f with permission over IRC from LegNeato. SeaMonkey will take this as a ride-along on our 2.0.8 oilspill release.

Comment 17

[(no longer active)]

(In reply to comment #16)
> I also re-imported this on the 1.9.1.12 relbranch as
> http://hg.mozilla.org/releases/mozilla-1.9.1/rev/1389736dbf5f with permission
> over IRC from LegNeato. SeaMonkey will take this as a ride-along on our 2.0.8
> oilspill release.

Great!  Maybe Thunderbird could do the same?

Comment 18

[Ludovic Hirlimann [:Usul]]

Verfifed Fixed with Mozilla/5.0 (Windows; U; Windows NT 5.1; fr; rv:1.9.2.9) Gecko/20100910 Thunderbird/3.1.4

Comment 20

[Al Billings [:abillings - ex-MoCo]]

Since Ludovic verified this for 1.9.2, adding the 'verified1.9.2' keyword.

Comment 21

[Al Billings [:abillings - ex-MoCo]]

Verified for 1.9.1 with Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.6; en-US; rv:1.9.1.12) Gecko/20100923 Shredder/3.0.9pre. The face attribute is preserved now in the signature.