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

Sublime: 'transpose' behaviour difference #16633

Closed
weinand opened this issue Dec 6, 2016 · 4 comments
Closed

Sublime: 'transpose' behaviour difference #16633

weinand opened this issue Dec 6, 2016 · 4 comments
Assignees
Labels
verification-needed Verification of issue is requested verified Verification succeeded
Milestone

Comments

@weinand
Copy link
Contributor

weinand commented Dec 6, 2016

while testing #16545:

VS Code's 'transpose' command doesn't work at end of the line. This behaviour differs from Sublime.

@rebornix
Copy link
Member

rebornix commented Dec 6, 2016

@weinand can you help provide more info about the behavior of Sublime Text? From what I see, both Code and Sublime do nothing when the cursor is at the end of the line

Sublime Text
sublime-transpose

Our implementation
vsc-tranpose

@rebornix
Copy link
Member

rebornix commented Dec 6, 2016

Maybe I know how to find the issue, if there are trailing spaces after the last nonwhitespace character, sublime text does nothing when the cursor is between the last nonwhitespace char and the space. However here we are still transposing.

I don't know why Sublime Text does it this way as I don't see any reason that the trailing spaces are special. Besides, Sublime Text 2 and Sublime Text 3 are doing Transpose differently. I'd like to keep it this way first and see more people's feedback :)

Thanks for pointing it out!

@rebornix rebornix added this to the January 2017 milestone Dec 6, 2016
@weinand
Copy link
Contributor Author

weinand commented Dec 6, 2016

@rebornix You tried transpose at the "end of the text".
What I meant was: when there is another line following, then transpose at the end of a line continues on the next line.

@rebornix
Copy link
Member

rebornix commented Dec 6, 2016

@weinand it's a valid one!

@rebornix rebornix added the verification-needed Verification of issue is requested label Dec 7, 2016
@rebornix rebornix modified the milestones: January 2017, November 2016 Dec 7, 2016
@weinand weinand added the verified Verification succeeded label Dec 7, 2016
@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
verification-needed Verification of issue is requested verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

2 participants