234638 - Graphical emoticon should show a tooltip with the textual emoticon

Status: VERIFIED FIXED

(SeaMonkey :: MailNews: Message Display, enhancement)

Description

[Tobias Burnus]

If 'Display emoticons as graphics' is shown, then :-) etc. are converted into
graphics. But sometimes one does want to see the original typeset thing (e.g. if
something is shown as emoticon which is not ment as one).

ChatZilla offers a tooltip showing the original typed text, unfortunally
Mail&News doesn't.

Comment 1

[Karsten Düsterloh]

Attachment: show html tooltips in messagepane, v1

I'm not quite sure if linking a navigator file inside mailnews is a good idea - otoh I don't know why browser.js is in navigator in the first place:
I'd expected it somewhere along browser.xml in global...

Anyway, when smilies have gotten a title attribute by bug 335344, this will show it as a tooltip.

Comment 2

[[e-mail]]

Comment on attachment 219774 [details] [diff] [review]
show html tooltips in messagepane, v1

From IRC: FillInHTMLTooltip (and possibly aHTMLTooltip) needs to move (probably to utilityOverlay.js) and the other two message windows also need to be patched.

Comment 3

[Mike Cowperthwaite]

*** Bug 318702 has been marked as a duplicate of this bug. ***

Comment 4

[Martijn Wargers (dead)]

(In reply to comment #2)
> (From update of attachment 219774 [details] [diff] [review])
> From IRC: FillInHTMLTooltip (and possibly aHTMLTooltip) needs to move (probably
> to utilityOverlay.js) and the other two message windows also need to be
> patched.

Does FillInHTMLTooltip still need to move to utilityOverlay.js?

Comment 5

[Karsten Düsterloh]

The fundamental problem (linking a navigator file inside mailnews) is still there, so this means most probably "yes". But I didn't look into this since then.
(Oh my, I really let that dangle for more than a year? :-( )

Comment 6

[Cédric "chewey" Menge]

Attachment: Show emoticon tooltip in message window and messenger.

This enables the tooltips for emoticons in both messenger.xul and MessageWindow.xul.

FillInHTMLTooltip() is moved from browser.js to a more generic location in contentAreaUtils.js - from where it still can be used by all necessary files.

Comment 7

[[e-mail]]

Comment on attachment 283928 [details] [diff] [review]
Show emoticon tooltip in message window and messenger.

The idea looks good but when I tried it it broke tooltips entirely in Editor (and note that Editor uses different tooltips anyway).

Comment 8

[Cédric "chewey" Menge]

Attachment: same as above + fixed editor.xul issue

Fixed editor issue by renaming the offending function from FillInHTMLTooltip() to FillInHTMLTooltipEditor()

Comment 9

[[e-mail]]

Comment on attachment 284354 [details] [diff] [review]
same as above + fixed editor.xul issue

>+  <tooltip id="aHTMLTooltip" onpopupshowing="return FillInHTMLTooltipEditor(this);"/>
For bonus points work out how to use "normal" tooltips for Composer's sidebar :-)

Comment 10

[Karsten Düsterloh]

Landed on trunk.

Many thanks, Chewey, for cutting this loose end here!