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)
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.
Comment 1•12 years ago
|
||
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.
Comment 2•12 years ago
|
||
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
Updated•12 years ago
|
Blocks: metrov1triage
Comment 3•12 years ago
|
||
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?
Updated•12 years ago
|
Whiteboard: [metro-mvp?]
Comment 4•12 years ago
|
||
(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.
Comment 5•12 years ago
|
||
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.
Component: Browser → General
OS: All → Windows 8 Metro
Hardware: All → x86
Version: 21 Branch → unspecified
Comment 6•12 years ago
|
||
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.
Updated•12 years ago
|
Component: General → Input
Version: unspecified → Trunk
Comment 7•12 years ago
|
||
On desktop Firefox, shift+right-click always shows the browser's built-in context menu.
Comment 8•11 years ago
|
||
I hit this problem with Google Drive on Nightly 20131124.
Comment 9•11 years ago
|
||
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.
Updated•11 years ago
|
Blocks: metrobacklog
Updated•11 years ago
|
No longer blocks: metrov2planning
Updated•11 years ago
|
Whiteboard: [defect]
Updated•11 years ago
|
Whiteboard: [defect] → [defect] p=0
Assignee | ||
Updated•10 years ago
|
OS: Windows 8 Metro → Windows 8.1
Comment 10•7 years ago
|
||
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.
Description
•