Closed Bug 1756323 Opened 3 years ago Closed 3 years ago

Tooltips position is incorrect when zoom in on the page

Categories

(Firefox :: General, defect, P3)

Product:
Firefox
Component:
General
Version:
Firefox 99
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
99 Branch
Tracking Flags:
Tracking Status
firefox-esr91 --- unaffected
firefox97 --- unaffected
firefox98 --- unaffected
firefox99 --- fixed

People

(Reporter: nayinain, Assigned: emilio)

References

(Regression)

Details

(Keywords: regression)

Attachments

(6 files)

Capture.webm
(deleted), video/webm
Details
Bug 1756323 - Deindent and simplify a bit ChromeTooltipListener::sTooltipCallback. r=smaug,jfkthame
(deleted), text/x-phabricator-request
Details
Bug 1756323 - Move ChromeTooltipListener out of a header file. r=smaug,jfkthame
(deleted), text/x-phabricator-request
Details
Bug 1756323 - Cleanup mouse event coord code. r=smaug,jfkthame
(deleted), text/x-phabricator-request
Details
Bug 1756323 - Fix tooltip coordinates in presence of full zoom. r=smaug,jfkthame
(deleted), text/x-phabricator-request
Details
Bug 1756323 - Test. r=jfkthame
(deleted), text/x-phabricator-request
Details
Attached video Capture.webm (deleted) —

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:100.0) Gecko/20100101 Firefox/100.0

Steps to reproduce:

  1. Zoom in (Ctrl - +) on the page.
  2. Hover the mouse over Mozilla logo on this page.

Actual results:

Tooltips position is incorrect.

Has Regression Range: --- → yes
Has STR: --- → yes
Keywords: regression
Regressed by: 1753836

Set release status flags based on info from the regressing bug 1753836

status-firefox97: --- → unaffected
status-firefox98: --- → unaffected
status-firefox99: --- → affected
status-firefox-esr91: --- → unaffected
Assignee: nobody → emilio
Flags: needinfo?(emilio)
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Flags: needinfo?(emilio)

No reason this can't live in the cpp file.

Depends on D139232

Depends on D139233

And cleanup similar code in nsXULTooltipListener. That code shouldn't
need to deal with zoom since the target and the tooltip are in the same
document.

Depends on D139234

Severity: -- → S3
Priority: -- → P3
Keywords: leave-open
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/7e6143b38d2c Deindent and simplify a bit ChromeTooltipListener::sTooltipCallback. r=smaug https://hg.mozilla.org/integration/autoland/rev/74fe022a83b9 Move ChromeTooltipListener out of a header file. r=smaug
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/d78e6cb7a956 Cleanup mouse event coord code. r=jfkthame
Attached file Bug 1756323 - Test. r=jfkthame (deleted) —
Keywords: leave-open
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/4937cfae71b6 Fix tooltip coordinates in presence of full zoom. r=jfkthame

https://hg.mozilla.org/mozilla-central/rev/d78e6cb7a956

Status: ASSIGNED → RESOLVED
Closed: 3 years ago
status-firefox99: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → 99 Branch
Component: DOM: Events → General
Product: Core → Firefox
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: