search

Zinggi / DictionaryAutoComplete

This adds dictionary entries to the completions inside comments. For lazy typers!
Other
94 stars 19 forks source link

ST3 compatibility #7

Closed wasabi-zz closed 11 years ago

wasabi-zz commented 11 years ago

Hi! Just wanted to know does this work on ST3?

Chris

Zinggi commented 11 years ago

very likely yes, although I haven't tested it yet. Someone just sent a pull request that should make it compatible ;)

wasabi-zz commented 11 years ago

Thanks! But doesn't seem to work for me.

Zinggi commented 11 years ago

Have you tried with the current version? I added another fix and it does work on my pc. (did you install with package control?) The problem was that the dictionary file is located under a different folder in ST3 and it's inside a zip file, so I had to unzip it first. If you're using a custom dictionary file, then it might not work yet. (because then my plugin wrongly assumes that your file is located inside a .sublime-package file, but it's probably not) EDIT: uploaded a quick fix for custom dictionary files, UNTESTED...

wasabi-zz commented 11 years ago

Yeah, I have EN_GB and EN_US installed. Don't know why. No, it doesn't work.

Zinggi commented 11 years ago

After you tried to make it work by pressing ctrl+space inside a comment, can you open the console and post it's output? Also, where is this dictionary file located? And your setting file, does it say: "dictionary": "Packages/Language - English/en_US.dic", or something else?

Zinggi commented 11 years ago

I think I found the issue. I was testing it with the portable version of ST3 and not the installed version. The folder structure is slightly different so it couldn't find the dictionary file. I'll try to fix that soon.

Zinggi commented 11 years ago

Ok, this should be fixed. Can you please validate?

wasabi-zz commented 11 years ago

Hi Florian,

Yes, it works! Thanks very much for looking into it!

Chris

On Sat, Nov 16, 2013 at 1:35 AM, Florian Zinggeler <notifications@github.com

wrote:

Ok, this should be fixed. Can you please validate?

— Reply to this email directly or view it on GitHubhttps://github.com/Zinggi/DictionaryAutoComplete/issues/7#issuecomment-28582846 .

Zinggi commented 11 years ago

great!