1285189 - HTMLTooltip useXulWrapper should default to false

Status: RESOLVED FIXED

(DevTools :: Shared Components, defect, P1)

Description

[Julian Descottes [:jdescottes]]

In bug 1267403, I implemented using a XUL panel wrapper for HTML Tooltips.

However after more testing yesterday I found two usability issues for which I don't have any easy fix for now. I will log separate bugs to track them, but in the meantime, I think they are severe enough to make the useXulWrapper option default to true.

Short summary of the issues:
- scroll events are disabled while the tooltip is opened and the mouse pointer is not on the tooltip
- mouseover and click events are not forwarded to devtools when the mouse pointer is on the filler space of the tooltip

Comment 1

[Julian Descottes [:jdescottes]]

Attachment: Bug 1285189 - change default value of HTMLTooltip:useXulWrapper to false;

Review commit: https://reviewboard.mozilla.org/r/62814/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/62814/

Comment 2

[Julian Descottes [:jdescottes]]

try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=78352f425bc6

Comment 3

[Julian Descottes [:jdescottes]]

Added a reference to the two bugs focusing on the issues specific to using the xul panel wrapper, Bug 1285206 & Bug 1285261

Comment 4

[Alexandre Poirot [:ochameau]]

Comment on attachment 8768728 [details]
Bug 1285189 - change default value of HTMLTooltip:useXulWrapper to false;

https://reviewboard.mozilla.org/r/62814/#review59744

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/fx-team/rev/6d33f8c374e8
change default value of HTMLTooltip:useXulWrapper to false;r=ochameau

Comment 6

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/6d33f8c374e8