sublime-elfinder icon indicating copy to clipboard operation
sublime-elfinder copied to clipboard

Updated for Sublime Text 3

Open me0wday opened this issue 8 years ago • 6 comments

Made the required changes to have this working on Sublime Text 3, including a workaround to an ST3 bug with the output regex feature.

Code could also be written to be cross compatible with ST2 but tried to keep it clean.

You have 3 branches too I see for each OS, where the submodule is part of the GIT, may need to do that again as ST3 does not pull down submodules :(

me0wday avatar Sep 14 '16 06:09 me0wday

Hi @thinkv, thank you kindly for this! I'll be happy to merge this in once I get a chance to test it. I don't think it's necessary for the code to be backwards compatible with ST2. I'd be happy for the ST2 package to keep the existing features but not be maintained. The ST3 package should be the main focus now. I will need to see if it's still necessary to have a separate branch for each OS in ST3. Its just calling a command line app so I wouldn't think it would be necessary. I'm currently on holiday so won't be able to test this out until October.

keeganstreet avatar Sep 14 '16 10:09 keeganstreet

@keeganstreet any news on that? Do you still planing test that?

ricardovsilva avatar May 09 '17 18:05 ricardovsilva

Sorry I don't really have time, and I don't use Sublime Text. Would anyone want to take over the Sublime Text Element Finder package?

keeganstreet avatar May 10 '17 10:05 keeganstreet

@keeganstreet I think that @thinkv may take that. If he is too busy I can take over that project. Best Regards.

ricardovsilva avatar May 10 '17 13:05 ricardovsilva

I will see what I can do tomorrow :)

me0wday avatar May 10 '17 14:05 me0wday

@ricardovsilva I have taken over at https://github.com/thinkv/sublime-elfinder

A PR has been made to wbond/package_control_channel to make the change over however I think they are quite slow in their merges. If you'd like to use/test it now, you can add my repo to your sublime and install until it is listed on the official list.

me0wday avatar May 18 '17 07:05 me0wday