CleanLinks icon indicating copy to clipboard operation
CleanLinks copied to clipboard

Not working anymore...

Open Greenpassion opened this issue 9 years ago • 19 comments

Firefox 48.0.2 / 64 / Multitask Not working anymore... Is it the 64 version or the multitask enabled? Thanks

Greenpassion avatar Sep 14 '16 06:09 Greenpassion

no e10 here, but extension doesn't work on 48.0.2

alekksander avatar Sep 14 '16 19:09 alekksander

No idea what you are talking about...

Firefox 48.0.2 x64 It is working at its best !

geokis avatar Sep 15 '16 03:09 geokis

sometimes it works… for example it's not possible to log in to aliexpress without adding it to the white list… but when i go to link from google, first it gives me: https://www.google.pl/url?sa=t&rct=j&q=&esrc=s&source=web&cd=13&ved=0ahUKEwiNyfqTipHPAhUJtxQKHVG6CvsQFghPMAw&url=https%3A%2F%2Fpl.wikipedia.org%2Fwiki%2FGrafika&usg=AFQjCNFmav2s9SazINQyZYOYlr2sVfNJmA&sig2=No-H5QAa6UY9rk_A6uKfYg&cad=rja and then redirects to: https://pl.wikipedia.org/wiki/Grafika Needless to say, links do not turn yellow (default highlight style [background:rgba(252,252,0,0.6); color: #000])

alekksander avatar Sep 15 '16 09:09 alekksander

Still not working... Removed and installed again...no results... Firefox 48.0.2 / 64 / MULTITASK

Greenpassion avatar Sep 20 '16 08:09 Greenpassion

Come on guys... Deactivate the experimental multiPROCESS option and the problem is gone...

See also #120

geokis avatar Sep 21 '16 00:09 geokis

please read carefully! i do not use multiprocess! that is: Multiprocess Windows 0/1 (Disabled)

alekksander avatar Sep 21 '16 11:09 alekksander

okay so i re-added button to the toolbar… and it worked again! removing the icon disables this extension. Filled out issue: https://github.com/diegocr/CleanLinks/issues/154

alekksander avatar Sep 21 '16 12:09 alekksander

Ok! So I disabled the multiPROCESS thing and it went back to normality... Thanks a lot!

Greenpassion avatar Sep 22 '16 05:09 Greenpassion

Adding the icon allowed the addon to work for me as well on Firefox 49.0.1.

Korpip avatar Sep 29 '16 00:09 Korpip

@geokis multiprocess isn't experimental anymore. It's in the stable release of Firefox. The fact it's being staged is because many add-on developers haven't made their add-ons multiprocess aware.

jawz101 avatar Sep 30 '16 20:09 jawz101

fwiw, in August i did started porting CL to be fully e10s compatible to an state that it's currently functional, however once it was made compatible i did realized an unfortunate fact, Firefox is unable to properly unload frame-scripts and any modules loaded through them, turning re-startless extensions into requiring a restart between upgrades, which is quite odd.

diegocr avatar Sep 30 '16 22:09 diegocr

Does that mean the e10s compatible build actually works - but it just means that a restart is needed to upgrade to a new version ?

Can we voluntarily "Beta Test" the working version anyway ?

(I have never understood the necessity or desire for restartless addons, I always restart after installing anything new anyway)

GitCurious avatar Oct 01 '16 01:10 GitCurious

Yes, that is what it means.

I can push what i've done till now, but first i need to clean up the code a bit, since is full of debugging stuff i've added while trying to deal with that.

diegocr avatar Oct 02 '16 03:10 diegocr

What's up with e10s support, having to restart is really no issue comparing to not being compatible at all..

piejanssens avatar Nov 15 '16 19:11 piejanssens

Any news on this? Having to restart is not a major problem and, maybe i am wrong but, not related to e10s validation

MikeZeDev avatar Dec 31 '16 16:12 MikeZeDev

I agree with piejanssens and MikeZeDev. Even if it needs a restart we still need it.

Quantum00 avatar Jan 07 '17 15:01 Quantum00

yes, please fix, it's getting urgent.

alekksander avatar Feb 19 '17 22:02 alekksander

Fwiw if it has to be e10s people can use Pure URL add-on. I think it does the same thing as this one

jawz101 avatar Feb 24 '17 04:02 jawz101

unfortunately it does not.

alekksander avatar Feb 24 '17 08:02 alekksander