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

Unhilight current line when in "edit mode" #43

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sampumon
Copy link

Unhilight any trailing space on current line(s) when editing them. This is a compromise between the possible modes of include_current_line setting.

Might be a bit hacky with global dicts saving stuff, but noticed similar hackiness with other globals already present :)

Set "trailing_spaces_unhilight_editing_line": true to enable. Note that it does its thing only if trailing_spaces_include_current_line is true (which it is by default).

Fixes #42.

Set "trailing_spaces_unhilight_editing_line": true to enable.
Does its thing only if "trailing_spaces_include_current_line" is true.

Fixes SublimeText#42.
@rchl
Copy link
Member

rchl commented Mar 28, 2020

I had no clue what this option is supposed to do before looking at the issue. Which means the naming is not good.
But honestly, I think we could instead allow auto value for trailing_spaces_include_current_line to toggle that behavior. That seems cleaner and more understandable to me.

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.

Sanitize trailing_spaces_include_current_line
3 participants