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

WebExt: External editor support (read/write extension directory) #2513

Closed
silverwind opened this issue Jun 5, 2017 · 5 comments · May be fixed by #3143
Closed

WebExt: External editor support (read/write extension directory) #2513

silverwind opened this issue Jun 5, 2017 · 5 comments · May be fixed by #3143
Labels
bankruptcy Issues that have been closed for "bug bankruptcy". Usually very old non-active issues.
Milestone

Comments

@silverwind
Copy link
Contributor

The runtime API provides multiple ways to read files from the extension directory, so what is left to explore is how to write files to the extension directory.

I think it's worth exploring if runtime#getPackageDirectoryEntry and fileSystem#getWritableEntry can used to write files to the extension directory, but I fear that the permission system might interfere.

@silverwind
Copy link
Contributor Author

silverwind commented Jun 5, 2017

Turns out fileSystem is not available to extensions, only to apps. So with a read-only API, it might still be possible to implement editor support but things like auto-update won't be possible.

@arantius
Copy link
Collaborator

This is effectively impossible. The only thing I can imagine is to use native messaging and a custom written local binary. That will be a lot of work, at best.

@arantius arantius modified the milestone: Pony Aug 16, 2017
@yfdyh000
Copy link

FYI: https://violentmonkey.github.io/2017/03/14/How-to-edit-scripts-with-your-favorite-editor/

@monk-time
Copy link

@yfdyh000 Thanks, that is incredibly helpful! I didn't even think about running python -m http.server to serve my userscript folder on localhost while I'm editing them. Now it's only two clicks and ~3 sec delay between them to reload a script from disk.

The only step missing to make it truly convenient is the Violentmonkey feature mentioned in the article: a checkbox on the GM "Install script" page to monitor the file instead of closing it after installing. Even simply keeping that installation page open instead of closing it after hitting Install would be awesome.

@ghost ghost mentioned this issue Aug 21, 2019
esperecyan added a commit to esperecyan/greasemonkey that referenced this issue May 17, 2022
esperecyan added a commit to esperecyan/greasemonkey that referenced this issue May 25, 2022
to be able to use external editor.

Resolves greasemonkey#2513 , resolves greasemonkey#3048 , resolves greasemonkey#2666 , resolves greasemonkey#2710 , resolves greasemonkey#2738 , resolves greasemonkey#2737 , resolves greasemonkey#2736 , resolves greasemonkey#2735 , resolves greasemonkey#3098 , and resolves part of greasemonkey#2669 and greasemonkey#3115 .
esperecyan added a commit to esperecyan/greasemonkey that referenced this issue May 27, 2022
to be able to use external editor.

Resolves greasemonkey#2513 , resolves greasemonkey#3048 , resolves greasemonkey#2666 , resolves greasemonkey#2710 , resolves greasemonkey#2738 , resolves greasemonkey#2737 , resolves greasemonkey#2736 , resolves greasemonkey#2735 , resolves greasemonkey#3098 , and resolves part of greasemonkey#2669 and greasemonkey#3115 .
esperecyan added a commit to esperecyan/greasemonkey that referenced this issue Jun 17, 2022
@arantius arantius added the bankruptcy Issues that have been closed for "bug bankruptcy". Usually very old non-active issues. label Nov 2, 2023
@arantius arantius closed this as completed Nov 2, 2023
esperecyan added a commit to esperecyan/greasemonkey that referenced this issue Jan 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bankruptcy Issues that have been closed for "bug bankruptcy". Usually very old non-active issues.
Projects
None yet
4 participants