search

jgbishop / colt

Firefox add-on that makes it easy to copy link text and locations.
Other
10 stars 4 forks source link

Add electrolysis support #12

Open jgbishop opened 10 years ago

jgbishop commented 10 years ago

From Gary Shapiro:

Colt does not work with e10s enabled in Firefox. Nightly will be turning e10s on as the default in the next week or two. Any plans on getting an e10s version of Colt soon?

Additional information: Electrolysis - MozillaWiki

jgbishop commented 10 years ago

Here's another relevant article: Working with multiprocess Firefox - Mozilla | MDN

jgbishop commented 9 years ago

At the moment, the only broken place appears to be down in LoadLinkData(). If I find any others, I'll document them here.

MarkRH commented 8 years ago

Using version 2.6.7 in Firefox 49 and it's one of mine listed in the extensions.ini as not being multi-process compatible. Checking in with the various authors on their status. I found this extension recently and use it all the time. Thanks for all your efforts.

jgbishop commented 8 years ago

I have no plans on porting this to e10s, but if someone is willing to provide a pull request making the necessary changes, I'll be glad to merge it.

sebastienbarre commented 7 years ago

Love this extension, would be great if it could support e10s. Maybe some of us could throw a few $$ to help you out? Patreon?

sebastienbarre commented 7 years ago

@jgbishop is a PR is to be submitted, should it be based on the master, dev, or e10s branch?

jgbishop commented 7 years ago

@sebastienbarre Base it off of master, please. It looks like dev and e10s are behind the times, and the e10s branch was mostly just an experiment anyways.

sebastienbarre commented 7 years ago

@jgbishop Thx. I honestly have never developed a Firefox extension, so I'm not sure I'm the most qualified.

For whoever willing to try, check chitsaou/copy-as-markdown at gh-43. The copy-as-markdown extension does only a subset of what ColT does, but they are similar otherwise, and it was recently updated to support e10s.

sebastienbarre commented 7 years ago

I love this extension, but I'm going to pass, it's over my head. I forked it, tried to install it as a temporary extension, but Firefox didn't comply, complaining that it needs to be converted to a restartless extension beforehand. And that's before even converting it to support e10s....

Quantum00 commented 7 years ago

Please please add e10s support! This addon is amazingly useful!