Closed Bug 1328011 Opened 4 years ago Closed 3 years ago

New broken set of APIs doesn't account zoom level, therefore tooltips, splitters and context menus appear in the wrong places


(DevTools :: Framework, defect, P2)

49 Branch


(firefox50 wontfix, firefox51 wontfix, firefox52 fix-optional, firefox53 wontfix, firefox54 fix-optional)

Tracking Status
firefox50 --- wontfix
firefox51 --- wontfix
firefox52 --- fix-optional
firefox53 --- wontfix
firefox54 --- fix-optional


(Reporter: arni2033, Unassigned)




(Keywords: regression, steps-wanted)


(2 files)

>>>   My Info:   Win7_64, Nightly 52, 32bit, ID 20161001030430 (2016-10-01)
0. Open url   data:text/html,<img src="about:logo">
1. Open devtools (F12)
2. Increase zoom level in devtools (Ctrl+"=")
3. Click on iframes dropmarker

AR:  List of iframes appears detached from iframes button
ER:  List of iframes should appear near the button

This is regression from bug 1266419. Regression range:

"steps-wanted" keyword means that you should write 2 more scenarios mentioned in the summary
"regressionwindow-wanted" keyword means that you should find regression windows for all scenarios
Component: Untriaged → Developer Tools: Framework
No longer blocks: 1277113
No longer blocks: 1266419
Depends on: 1266419
Version: Trunk → 49 Branch
Seems worth fixing to me...  Honza, can you evaluate further?
Flags: needinfo?(odvarko)
Priority: -- → P2
The list of frames (drop down menu) coordinates are calculated here:

The logic seems to be wrong since it doesn't respect the current zoom level. I think that fixing this should fix the reported problem.

@Sébastien: Perhaps we could use win.devicePixelRatio to fix this?

Flags: needinfo?(odvarko) → needinfo?(amarok)
> @Sébastien: Perhaps we could use win.devicePixelRatio to fix this?

Yeap, I think we can (like I do in the attachment). But we need to check on a High DPI monitor (to avoid things like bug 1328895). If it doesn't work on a High DPI monitor, we should replace screenX, screenY and rect by value in physical screen pixels not in CSS pixels.
Flags: needinfo?(amarok)
Attachment #8826740 - Flags: feedback?(odvarko)
@ntim: can you please checkout the patch?

Flags: needinfo?(ntim.bugs)
Attached image Screenshot
Seems like the top position calculation is wrong.
Flags: needinfo?(ntim.bugs)
Comment on attachment 8826740 [details] [diff] [review]

Please see comment #5

@Sébastien, any progress on this?

Attachment #8826740 - Flags: feedback?(odvarko) → feedback-
Too late for a fix for 53, fix-optional for 54, minor carryover regression.
Product: Firefox → DevTools
Tested again on macOS with Firefox 65 (nightly) and this does not happen anymore. So I'm going to close this for now.
Closed: 3 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.