search

vyznev / chatjax

ChatJax++, a user script to enable MathJax in Stack Exchange chat
ISC License
7 stars 3 forks source link

Does not work in FireFox 107 + GreaseMonkey #4

Open Anixx opened 1 year ago

Anixx commented 1 year ago

Works well with PaleMoon, but not with Firefox.

makyen commented 1 year ago

As written, this userscript is not compatible with Greasemonkey 4.X (4.X is required to use Greasemonkey in Firefox, as of years ago), but is compatible with Greasemonkey 3.X, which I'd expect to work in Pale Moon.

The Greasemonkey developers made the intentional choice to make Greasemonkey 4.X incompatible with a large percentage of existing userscripts which had been written for Greasemonkey 3.X. Because of these intentional choices, you will find that a large percentage of userscripts (e.g. 75%) won't work in Greasemonkey 4.X,. From a user's perspective, it will be quite difficult to see if there is full compatibility. How broken things are will be different for each script (e.g. partial functionality, erroneous actions, etc.).

For the vast majority of users, I recommend against using Greasemonkey 4.X in Firefox, due to the incompatibilities which will crop up with lots of userscripts. Overall, users will just have a better experience with one of the other userscript managers (e.g. Tampermonkey or Violentmonkey).

Anixx commented 1 year ago

Thank you very much, with Violentmonkey works well.

Anixx commented 1 year ago

Closing

makyen commented 1 year ago

@Anixx It's unclear to me if this issue should remain open or be closed. This issue is accurate. The userscript is incompatible with Greasemonkey 4.X. The userscript could be modified to be compatible with Greasemonkey 4.X. I'd only close this if there's an explicit direction from the developer that Greasemonkey 4.X compatibility would be refused (i.e. that any PR with code that resolves the incompatibility would be closed as "won't be merged").

While I can certainly understand that there won't be effort by the developer put into making it compatible – I've, effectively, made that choice on at least one userscript – that doesn't mean that other people shouldn't know that it is known to be incompatible (i.e. that this is an actual issue) and that code to resolve it could be submitted as a PR.

Anixx commented 1 year ago

Okay, let it stay open.