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

Fix add selection above and add selection below hotkey on Ubuntu #51

Closed
Tyriar opened this issue Nov 18, 2015 · 7 comments
Closed

Fix add selection above and add selection below hotkey on Ubuntu #51

Tyriar opened this issue Nov 18, 2015 · 7 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug help wanted Issues identified as good community contribution opportunities verified Verification succeeded
Milestone

Comments

@Tyriar
Copy link
Member

Tyriar commented Nov 18, 2015

I'm used to alt+shift+up and alt+shift+down expanding the selection above and below the current anchor, as far as I can see this is not implemented in core or an extension.

Edit: This is actually implemented and working fine, the hotkey ctrl+alt+up/down just conflicts with the Ubuntu OS hotkey to change workspace.

@jrieken jrieken added the feature-request Request for new features or functionality label Nov 18, 2015
@jel-massih
Copy link

Is this different than the current add cursor above/below with ctrl+alt+up and ctrl+alt+down?

@pali88
Copy link

pali88 commented Nov 22, 2015

Try

@Tyriar Tyriar changed the title Implement add selection above and add selection below Fix add selection above and add selection below hotkey on Ubuntu Nov 22, 2015
@alexdima alexdima self-assigned this Nov 23, 2015
@Tyriar
Copy link
Member Author

Tyriar commented Nov 23, 2015

alt+shift+up actually conflicts as well, which conflicts on atom's defaults.

@alexdima
Copy link
Member

@Tyriar I only use Linux in a VM. I think for me some keybindings are lost due to the VM runner.

If you run Linux natively, I would love some help to figure out what would be better default keybindings for editor.action.insertCursorAbove and editor.action.insertCursorBelow:

{ "key": "ctrl+meta+up",          "command": "editor.action.insertCursorAbove",
                                     "when": "editorTextFocus" },
{ "key": "ctrl+meta+down",        "command": "editor.action.insertCursorBelow",
                                     "when": "editorTextFocus" },

@Tyriar
Copy link
Member Author

Tyriar commented Nov 24, 2015

@alexandrudima I'm used to alt+shift+down as the default for down (as on atom) and I guess I don't use the insert up because alt+shift+up conflicts as well. This is specific to Ubuntu, not sure about other distros. It would seem that we're running out of modifier key combinations. I'll see what Sublime uses by default tomorrow.

@alexdima alexdima added bug Issue identified by VS Code Team member as probable bug help wanted Issues identified as good community contribution opportunities and removed feature-request Request for new features or functionality labels Nov 24, 2015
@Tyriar
Copy link
Member Author

Tyriar commented Nov 24, 2015

@alexandrudima alt+shift+down/up is the default on Atom and Sublime, makes sense to follow suite.

@alexdima
Copy link
Member

I've pushed the following change to the defaults in c96af39 (tried on ubuntu and lubuntu):

  • move lines: from ctrl+down/up to alt+down/up (aligns with default on windows)
  • add cursor: from ctrl+win+down/up to shift+alt+down/up and also ctrl+shift+down/up

This frees up ctrl+up/down also on linux (it is already free on windows) for a future scroll viewport commands.

@alexdima alexdima added this to the Dec 2015 milestone Dec 16, 2015
@alexdima alexdima assigned aeschli and unassigned alexdima Dec 16, 2015
@aeschli aeschli added the verified Verification succeeded label Dec 17, 2015
@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
bug Issue identified by VS Code Team member as probable bug help wanted Issues identified as good community contribution opportunities verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

6 participants