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

words don't get replaced #6

Open
rpietro opened this issue Jun 21, 2015 · 4 comments
Open

words don't get replaced #6

rpietro opened this issue Jun 21, 2015 · 4 comments

Comments

@rpietro
Copy link

rpietro commented Jun 21, 2015

using the latest version of the plugin and sublime 3. when I do the search the menu shows me all options, but when I select one it doesn't replace the current word.

btw, love your plugin, extremely useful

@alimuldal
Copy link

I'm experiencing the same issue (Sublime Text 3 build 3083, Thesaurus v2014.11.25.15.06.12)

@cvladan
Copy link

cvladan commented Dec 11, 2015

Also for me.

File "C:\Program Files\Sublime Text 3\sublime.py", line 685, in replace
    raise ValueError("Edit objects may not be used after the TextCommand's run method has returned")
ValueError: Edit objects may not be used after the TextCommand's run method has returned

@LaryLee
Copy link

LaryLee commented Dec 14, 2015

Replace in File thesaurus.py

self.view.replace(self.edit, self.region, value.strip().lower())

with

self.view.run_command("insert", {"characters": value.strip().lower()})

At this point the edit object is not active anymore, so you have to run the replace with a command

Kristinita pushed a commit to Kristinita/Thesaurus that referenced this issue Jan 6, 2017
See [**LaryLee comment**](Nucc#6 (comment)). It's worked for me.
@Kristinita Kristinita mentioned this issue Jan 6, 2017
@rpietro rpietro changed the title words doesn't get replaced words don't get replaced Feb 26, 2017
@gmalone
Copy link

gmalone commented Sep 19, 2018

Installed from Package Control (2018.09.15), Thesaurus works generally, but (still) doesn't replace the selected word in the text body. I see that @Kristinita fixed it, but apparently not integrated into the main download, no?

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

No branches or pull requests

5 participants