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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Copy in chrome dev tools not working #55147

Closed
isidorn opened this issue Jul 26, 2018 · 3 comments
Closed

Copy in chrome dev tools not working #55147

isidorn opened this issue Jul 26, 2018 · 3 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug macos Issues with VS Code on MAC/OS X verified Verification succeeded
Milestone

Comments

@isidorn
Copy link
Contributor

isidorn commented Jul 26, 2018

Only reproduces on mac. Works on win and linux.

  1. F1 > developer tools
  2. Try to copy any text via cmd + c
  3. Text is not copied 馃悰

Started appearing about the time we merged the menu work so our suscipion is that is the cause.
Also note that when doing cmd + c, chrome dev tools is expecting there is a copy command under the edit menu (which we do have)

Assigning to July since this is hurting a bit of team's productivity.

fyi @jrieken

@isidorn isidorn added bug Issue identified by VS Code Team member as probable bug macos Issues with VS Code on MAC/OS X workbench-menu labels Jul 26, 2018
@isidorn isidorn added this to the July 2018 milestone Jul 26, 2018
@sbatten
Copy link
Member

sbatten commented Jul 26, 2018

Ah, this happens because on macOS, you must register the role of the action in the menubar. In the old code it was done statically like so https://github.com/Microsoft/vscode/blob/1dfc5e557209371715f655691b1235b6b26a06be/src/vs/code/electron-main/menus.ts#L548

This is only done for cut, copy, paste and on macOS so we could just add a function to menubar.ts to check the actionid and add the role. otherwise, the menuregistry would need to be extended, but I don't see that as worth it. What do you think?

@isidorn
Copy link
Contributor Author

isidorn commented Jul 27, 2018

@sbatten great thanks for the explanation. Yeah let's solve it like that, it is a bit of a special action.

@isidorn
Copy link
Contributor Author

isidorn commented Jul 27, 2018

Thanks, this now works great -> verified

@isidorn isidorn added the verified Verification succeeded label Jul 27, 2018
@vscodebot vscodebot bot locked and limited conversation to collaborators Sep 10, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug macos Issues with VS Code on MAC/OS X verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

2 participants