Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow users to re-enable browser-native contextmenu #2301

Closed
tmcw opened this issue Mar 21, 2016 · 4 comments
Closed

Allow users to re-enable browser-native contextmenu #2301

tmcw opened this issue Mar 21, 2016 · 4 comments

Comments

@tmcw
Copy link
Contributor

tmcw commented Mar 21, 2016

mapbox-gl-js version:

Steps to Trigger Behavior

  1. Optionally call map.off('contextmenu')
  2. Right-click on the map.

Expected Behavior

The browser's context menu will appear.

Actual Behavior

The _onContextMenu handler still prevents the event from having a default action.

@lucaswoj
Copy link
Contributor

This came up in user testing. It is frustrating for new users to be unable to open devtools on a page with a full screen map. Our example code creates a fullscreen map.

@lucaswoj lucaswoj changed the title Ability to re-enable browser-native contextmenu Allow users to re-enable browser-native contextmenu Jul 29, 2016
@mourner
Copy link
Member

mourner commented Sep 13, 2016

It is frustrating for new users to be unable to open devtools on a page with a full screen map.

Other ways to open a devtools page (just in case anyone is struggling with the above):

  • Options (on the right of the url tab) -> More Tools -> Developer Tools
  • View -> Developer -> Developer Tools
  • Option-Command-J

@jplante
Copy link

jplante commented Sep 21, 2018

So right now the Mapbox Map does not allow right-clicking to open up the context-menu?

@initialdmg
Copy link

While this will make preserveDrawingBuffer invalid. If dragRotate is disabled, the context menu will display when click on the map, which indicates user to save the picture of the map.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants