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

Select matching delimiter should be limited to just the delimiter #2367

Closed
Naatan opened this Issue Feb 23, 2017 · 3 comments

Comments

Projects
None yet
3 participants
@Naatan
Member

Naatan commented Feb 23, 2017

Short Summary

When this feature triggers it basically starts a multi-cursor selection and then just lets multi-cursor handle everything. This makes for annoying use-cases where starting a regular selection turns into a multi-selection.

Steps to Reproduce

Create a JS file with contents:

console.log("foo"|)

| represents your caret

Press shift+left twice

Expected results

The first iteration should select the delimiters, the second should revert to single select because obviously you're no longer intending to select the delimiter if your selection expands beyond delimiters.

Actual results

It selects the delimiters as well as the characters in front of them.

@Naatan Naatan added the Type: Bug label Feb 23, 2017

@Naatan Naatan added this to the 10.2.1 milestone Feb 23, 2017

@sfsccn

This comment has been minimized.

sfsccn commented Feb 23, 2017

I'm probably experiencing the same thing. I just upgraded to 10.2 and if I shift-select quoted text using arrow keys to add to the selection I get two cursors (weird) that keep selecting more than I want. It seems to depend on the language I'm viewing as.

Reproduce:

  • Edit new my-file.txt
  • Insert "Here is some 'quoted text'"
  • View as language text and shift select end of line: nothing
  • View as MySQL: dual cursor thing
  • View as JavaScript: dual cursor thing
@Naatan

This comment has been minimized.

Member

Naatan commented Feb 23, 2017

Fyi for now you can just turn this off via

Preferences > Editor > Smart Editing > Edit matching delimiter on select

@Naatan

This comment has been minimized.

Member

Naatan commented Feb 23, 2017

Prioritizing this as it can be quite annoying at the moment. If the fix for this is not feasible for 10.2.1 then we should default this pref to be turned off for 10.2.1.

mitchell-as added a commit that referenced this issue Feb 27, 2017

editor: Collapse multiple selection when selecting beyond matched del…
…imiter - fixes #2367

rn=

(integrated from the KomodoIDE 10.2.x branch change 257cb8f by Mitchell <mitchellb@activestate.com>)

Komodo/KomodoIDE@257cb8f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment