|Submitter||Diff||Changes||Open Issues||Last Updated|
|Error loading review requests:|
Created attachment 8898902 [details] context-menu.png STEPS TO REPRODUCE 1. Be syncing at least two devices 2. Open Page Action Menu 3. Open Send Page to Device 4. Right-click on device name EXPECTED RESULTS - Nothing happens ACTUAL RESULTS - Add to Address context menu item appears - Selecting it does nothing
I think this should be as simple as moving the context/contextmenu attribute to the main view of the page action popup.
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Iteration: --- → 57.2 - Aug 29
Priority: P3 → P1
When fixing this I realized you could context-click on empty bits of the main view in the panel and we'd show the context menu and it wouldn't do anything, so I fixed that too. :-)
Comment on attachment 8900241 [details] Bug 1391705 - don't show context menu except on toolbarbuttons on the main view, https://reviewboard.mozilla.org/r/171628/#review176988 Thank you
Attachment #8900241 - Flags: review?(adw) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/4f0c2d08aa32 don't show context menu except on toolbarbuttons on the main view, r=adw
Status: ASSIGNED → RESOLVED
Last Resolved: 11 months ago
status-firefox57: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
I have reproduced the issue mentioned in comment 0 using an affected Firefox 57.0a1 build (BuildId:20170818100226). I have verified that the issue is not reproducible using Firefox 57.0b7 (Build Id:20171009192146) on Windows 10 64bit, macOS 10.11.6 and Ubuntu 16.04 64bit.
Status: RESOLVED → VERIFIED
status-firefox57: fixed → verified
You need to log in before you can comment on or make changes to this bug.