search

SublimeText / WordHighlight

Highlight all copies of the currently selected word.
MIT License
180 stars 24 forks source link

Delay on highlighting #4

Closed ghost closed 13 years ago

ghost commented 13 years ago

Put delay on highlighting: prevent whole code from executing, when user is in process of selecting text.

Do not retrieve settings on every call, but only once.

The patch also partially solves the current text selection glitch, caused, apparently, by view.add_regions being called multiple times in a row.

adzenith commented 13 years ago

I don't like the new way the settings are read because now you have to re-open Sublime Text if you change the settings. Can we change it back?

ghost commented 13 years ago

I'm sorry, it is my fault — I simply forgot settings won't be re-read that way. I have that part restored. As a whole, the patch needs more testing, I had it pushed somewhat in a hurry because of the discussion on sublime forums, starting with http://www.sublimetext.com/forum/viewtopic.php?f=2&t=3388&start=10#p15974 (across 3 pages) — expecting more user feedback