Closed Bug 874963 Opened 11 years ago Closed 11 years ago

Work - popups don't move when scrolling

Categories

(Firefox for Metro Graveyard :: Input, defect, P2)

x86_64
Windows 8.1
defect

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 24

People

(Reporter: rsilveira, Assigned: jwilde)

References

Details

(Whiteboard: [shovel-ready] feature=work)

Attachments

(2 files)

Attached image Screenshot
See attached screenshot. We should dismiss when scrolling.

Desktop dismisses when scrolling by mouse but it moves with the page when scrolling by touch.

Points estimate=1
No longer blocks: metrov1defect&change
Summary: defect - popups don't move when scrolling → Work - popups don't move when scrolling
Whiteboard: feature=work
I think this is p1 if we dismiss in both cases.
Whiteboard: feature=work → [shovel-ready] feature=work
Priority: -- → P2
Assignee: nobody → hello
Blocks: metrov1it9
No longer blocks: metrov1it9
Status: NEW → ASSIGNED
Attached patch patch v1Splinter Review
Adds an event handler that closes the menu ui when the user starts to pan.
Comment on attachment 761817 [details] [diff] [review]
patch v1

Review of attachment 761817 [details] [diff] [review]:
-----------------------------------------------------------------

per IRC, sending this to fryn since he's new firehose person?
Attachment #761817 - Flags: review?(fyan)
Comment on attachment 761817 [details] [diff] [review]
patch v1

Review of attachment 761817 [details] [diff] [review]:
-----------------------------------------------------------------

:D
Attachment #761817 - Flags: review?(fyan) → review+
https://hg.mozilla.org/mozilla-central/rev/d5917b4a5479
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 24
OS: Windows 8 Metro → Windows 8.1
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: