remove tooltip for emoticons pref

VERIFIED FIXED in mozilla1.0

Status

--
minor
VERIFIED FIXED
18 years ago
14 years ago

People

(Reporter: dean_tessman, Assigned: stephend)

Tracking

Trunk
mozilla1.0

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Reporter)

Description

18 years ago
There's a tooltip for the "Display emoticons as graphics" pref checkbox in Mail
& Newsgroups > Message Display, which reads "Convert emoticons".  Checkboxes
shouldn't have tooltips.  Die tooltip die!
->mailnews. reassign as needed!
Assignee: sgehani → sspitzer
Component: Preferences → Mail Window Front End
Product: Browser → MailNews
QA Contact: sairuh → esther
I can fix this, but I'll need to make sure it's to spec to remove this.
Created attachment 52087 [details] [diff] [review]
The fix, simply remove the entity in the DTD, and the attribute of the XUL element.
Taking bug.
Assignee: sspitzer → stephend
accepting.
Status: NEW → ASSIGNED
Keywords: patch, review
Target Milestone: --- → mozilla1.0
(Reporter)

Comment 6

18 years ago
I can't see how having a tooltip on a checkbox could be to any sort of spec.  If
it is, the spec's wrong.  Given that this is the only tooltip in the .dtd, I
assume it's a leftover a bad idea from the past.

Comment 7

18 years ago
stephen, go for it. Thanks. :-)
(Reporter)

Comment 8

18 years ago
Comment on attachment 52087 [details] [diff] [review]
The fix, simply remove the entity in the DTD, and the attribute of the XUL element.

in that case, r=dean_tessman@hotmail.com
Attachment #52087 - Flags: review+

Comment 9

18 years ago
Comment on attachment 52087 [details] [diff] [review]
The fix, simply remove the entity in the DTD, and the attribute of the XUL element.

sr=mscott since jglick approves.
Attachment #52087 - Flags: superreview+

Updated

18 years ago
QA Contact: esther → olgam
f-i-x-e-d
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
(Reporter)

Comment 11

18 years ago
y-e-s i-t i-s
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.