Panels such as those of the Inspector jump around when dragged across multiple monitors

RESOLVED FIXED

Status

()

Core
XP Toolkit/Widgets: XUL
RESOLVED FIXED
7 years ago
6 years ago

People

(Reporter: Neil Deakin (mostly unavailable until September), Assigned: Neil Deakin (mostly unavailable until September))

Tracking

Trunk
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(blocking2.0 -)

Details

Attachments

(1 attachment)

Created attachment 469153 [details] [diff] [review]
fix

This is because panels are constrained to the area of the screen they are on, so jump to fit when crossing over.

Instead, when moving a popup, don't constrain or flip a popup.
haven't tested this on windows but presumably the behavior's the same there. Thanks for filing!
OS: Mac OS X → All
Hardware: x86 → All
this is a usability improvement. Should this block?
blocking2.0: --- → ?
Comment on attachment 469153 [details] [diff] [review]
fix

This patch skips the block where we constrain and flip the panel to fit on the screen when the panel is being moved. There might be specific cases where someone does want this behaviour, for instance, when popup.moveTo is called directly but I can add that in a followup bug.
Attachment #469153 - Flags: review?(roc)
Attachment #469153 - Flags: review?(roc) → review+
Not going to hold the release for this change. Would definitely consider approval if the patch is safe and tested.
blocking2.0: ? → -
http://hg.mozilla.org/mozilla-central/rev/3de085749863
Status: ASSIGNED → UNCONFIRMED
Ever confirmed: false
Flags: in-testsuite+
Status: UNCONFIRMED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.