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

delete completions.py - removing . from word_separators leads to unexpec... #5

Merged
merged 1 commit into from
Jun 3, 2014

Conversation

MattDMo
Copy link
Contributor

@MattDMo MattDMo commented Jun 2, 2014

Removing . from word_separators leads to unexpected and unwanted behavior - when double-clicking on Property in either of the two lines of code below

Object.Property.field;
Object.Property:FunctionName();

when using Sublime Text 2, all of the first line up to the ending semi-colon ; is selected, and in the second line Object.Property is selected. This is not expected behavior in keeping with Sublime's actions when using other languages/syntaxes, and should be removed. This shouldn't be necessary to assist with completions.

This pull request is in response to this question on StackOverflow.

szensk added a commit that referenced this pull request Jun 3, 2014
delete completions.py - removing . from word_separators leads to unexpec...
@szensk szensk merged commit 1082636 into szensk:master Jun 3, 2014
@szensk
Copy link
Owner

szensk commented Jun 3, 2014

I added it such that "love.graphics.pri" followed by completion wouldn't lead to "love.graphics.love.graphics.print(...)". It was, however, a bad idea since most people likely do not want that behavior. Secondly it's easier to set with syntax specific settings. Not sure what I was thinking with this. 👍

@MattDMo
Copy link
Contributor Author

MattDMo commented Jun 3, 2014

No worries. Thanks for being so responsive!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants