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

editor.multicursorModifier not discoverable (add toggle menu item) #27551

Closed
isidorn opened this issue May 30, 2017 · 4 comments
Closed

editor.multicursorModifier not discoverable (add toggle menu item) #27551

isidorn opened this issue May 30, 2017 · 4 comments
Assignees
Labels
feature-request Request for new features or functionality verification-needed Verification of issue is requested verified Verification succeeded
Milestone

Comments

@isidorn
Copy link
Contributor

isidorn commented May 30, 2017

Refs: #27480

The setting editor.multicursorModifier is not that discoverable, I as a user would not even think about looking there. In order to impove this I would consider adding a toggle action "Use ctrl + click to add cursor / use altr + click to add cursor" in the selection menu which already has all the multi cursor actions.

Without that the only way users find it is via googling around / stack overflow

@alexdima alexdima added this to the May 2017 milestone May 31, 2017
@alexdima alexdima added the feature-request Request for new features or functionality label May 31, 2017
@alexdima alexdima changed the title editor.multicursorModifier not discoverable editor.multicursorModifier not discoverable (add toggle menu item) May 31, 2017
@alexdima
Copy link
Member

alexdima commented Jun 1, 2017

Suggestion from @seanmcbreen in #27503 (comment)

Place a toggle in the Selection menu for enabling this - I propose:
Use Ctrl Click for Multi-cursor and Use Alt Click for Multi-cursor
Make this it's own section at the bottom (less convicted on where it is just trying to propose something)

@alexdima alexdima added the verification-needed Verification of issue is requested label Jun 1, 2017
@bpasero bpasero added the verified Verification succeeded label Jun 2, 2017
@bpasero
Copy link
Member

bpasero commented Jun 2, 2017

Seems to work, it is not a separate section though:

image

@isidorn
Copy link
Contributor Author

isidorn commented Jun 2, 2017

Yeah I believe the multicursors deserve their section and this should be the last entry in that section

@alexdima
Copy link
Member

alexdima commented Jun 2, 2017

All those actions are about multicursor, so that's why I thought this should go with them. I am open to changing this, I don't have strong feelings either way.

@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request Request for new features or functionality verification-needed Verification of issue is requested verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

3 participants