-
Notifications
You must be signed in to change notification settings - Fork 31
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
Edit button with latest Firefox due to e10s #80
Comments
Related to #79 I think. |
I'm going to close this as a duplicate. |
@docwhat I don't think it's related to #79. Or at least, this one is not a duplicate at all. On the one hand, the missing edit button I described in that issue occurred with e10s disabled. (As I couldn't even enable it in FF 47.) On the other hand, in #79 the edit button was missing only while trying to edit a Stylish theme, not the small button that appears in the corner of text boxes on any web page. By the way, I can confirm that I have the same issue as @Jereviendrai on FF 49.0.2 when e10s is on (by adding |
Sorry for disappearing for so long. I just wanted to add that I don't use the Stylish addon, so I don't see how this is related to #79. |
I take it back, I don't think it is related to #79... I'm not sure what's up then. |
On Firefox 50.0 with e10 the button does appear if I open the inspector, but until that it is hidden. |
https://addons.mozilla.org/en-US/firefox/addon/withexeditor/ works with e10s, for those interested in switching and/or for anyone interested in patching It's All Text for those that want to use emacsclient with WithExEditor, the combination that worked for me was to check "Put File Path After Command Line Options", leave unchecked "Run Command in A Shell", and set this .sh file as the editor: #!/bin/bash |
@braham-snyder Thank you for the suggestion! So far it runs well with Neovim-Qt, too. I used the following in an .sh file:
|
withExEditor looks pretty nice, though it requires the user to setup Node.js and run a host service. It was how I was thinking of doing the next version, but using a go binary or something similar I don't know how to fix the e10s "edit button" problem but would love any pull requests. Alternatively, someone could write a portable version of withExEditorHost and make withExEditor more simple to use for "normal" people. |
Just FYI, I wrote an extension with similar base functionality called Textern. It uses purely WebExtensions and should thus be fully compatible with e10s! |
@jlebon: Cool! I made several attempts at doing something like that, but I kept getting lost. If you speak I'm looking for someone to take over IAT. Would you be interested? We could create a new branch ( Would you be willing? |
@docwhat Rewriting in Unfortunately, I don't have the bandwidth and willpower right now to invest much more time in the project beyond simple bugfixes and accepting patches. It was more of a way to exercise my JS and Python while scratching an itch. :) I wouldn't feel comfortable making any commitment without the drive to follow through with them. Hope you understand! |
No worries, @jlebon. I'm in a similar boat with IAT itself. I've run out of steam on it. |
In Firefox Nightly (Version 51.0), when e10s is enabled (preferences > general > enable multi-process Nightly), the edit button does not appear when first opening a webpage. To make it appear, I need to switch to a different tab and then switch back.
The keyboard shortcut doesn't work either until I switch back and forth between tabs.
The plugin works normally when e10s is disabled in the preferences.
The text was updated successfully, but these errors were encountered: