Closed Bug 842203 Opened 12 years ago Closed 7 years ago

Web sites that trap the contextmenu event stop Metro Firefox app bars from appearing

Categories

(Firefox for Metro Graveyard :: Input, defect)

x86
Windows 8.1
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INCOMPLETE

People

(Reporter: dhunterz, Unassigned)

References

Details

(Whiteboard: [defect] p=0)

User Agent: Mozilla/5.0 (Windows NT 5.1; rv:21.0) Gecko/20130213 Firefox/21.0 Build ID: 20130213040206 Steps to reproduce: 1. visit "http://1967jk.blog.me/" 2. right-click they have javascript called "disable right-click" from http://t.static.blog.naver.net/mylog/versioning/JindoComponent-140112.js Actual results: nothing. Expected results: opened menu.
Sorry, I don't understand why this is a bug. The web site intentionally prevents the context menu from appearing (by using preventDefault on the contextmenu event), and as far as I can tell this is working in Metro Firefox the same way it works in desktop Firefox and other browsers.
Oh, now I see -- the expected result is that the Metro Firefox toolbars should appear, but this does happen because the web site stops propagation of the contextmenu event. I thought we had an open bug about this, but I can't find it now...
Status: UNCONFIRMED → NEW
Ever confirmed: true
Summary: right click influenced javascript on specific website → Web sites that trap the contextmenu event stop Metro Firefox app bars from appearing
Blocks: 831934
Whiteboard: [metro-mvp?]
Can we simply disable that feature for now? It's a user pref so I imagine it's not difficult to toggle and the upsid of the semi-useful web feature seem far outweighed by the downside of it breaking a fundamental Windows gesture. What does IE do here?
Whiteboard: [metro-mvp?]
No longer blocks: 831934
Blocks: 831899
Blocks: 831934
Blocks: 845152
No longer blocks: 831899
(In reply to Asa Dotzler [:asa] from comment #3) > Can we simply disable that feature for now? It's a user pref so I imagine > it's not difficult to toggle and the upsid of the semi-useful web feature > seem far outweighed by the downside of it breaking a fundamental Windows > gesture. What does IE do here? IE has the same behavior, nothing happens when you right-click - no appbar.
Thanks, Rodrigo. This may be difficult to solve. Given that IE's done nothing there and that we don't have an obvious solution in hand and given that the touch gesture is a solid workaround, I think we push this out to a later version.
Blocks: metrov2planning
No longer blocks: 831934, metrov1triage, 845152
Component: Browser → General
OS: All → Windows 8 Metro
Hardware: All → x86
Version: 21 Branch → unspecified
A keyboard workaround is the shortcut win+z, which is the default shortcut for showing the app bar. Currently SHIFT+F10 and the keyboard context menu button have the same effect of showing the app bar, but that's probably a bug.
Component: General → Input
Version: unspecified → Trunk
On desktop Firefox, shift+right-click always shows the browser's built-in context menu.
I hit this problem with Google Drive on Nightly 20131124.
The easiest solution is to make this event listener capture (change "false" to "true" on this line): http://hg.mozilla.org/mozilla-central/file/757c2011df5b/browser/metro/base/content/contenthandlers/ContextMenuHandler.js#l15 This would completely break the ability of web pages to receive or prevent "contextmenu" events, but based on comment 5 it sounds like we are willing to make that trade-off.
No longer blocks: metrov2planning
Whiteboard: [defect]
Whiteboard: [defect] → [defect] p=0
Depends on: 966411
OS: Windows 8 Metro → Windows 8.1
Mass close of bugs in obsolete product https://bugzilla.mozilla.org/show_bug.cgi?id=1350354
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.