Review by Lampe2020
Rated 5 out of 5
ruma Lampe2020, 4 moannen lynI don't know why this isn't core functionality in Firefox and why this extension isn't among Mozilla's "Recommended" either.
This extension is an immediate improvement, because I now have a much easier time understanding source files that I open in Firefox.
Some slight changes for this to be perfect would be to enable autodetect by default and to choose a more contrasty colour scheme by default (Qt Creator Light/Dark maybe) so the highlighted text doesn't fade into the background.
This extension is an immediate improvement, because I now have a much easier time understanding source files that I open in Firefox.
Some slight changes for this to be perfect would be to enable autodetect by default and to choose a more contrasty colour scheme by default (Qt Creator Light/Dark maybe) so the highlighted text doesn't fade into the background.
17 reviews
- Rated 5 out of 5ruma MP3Martin, 9 moannen lyn
- Rated 5 out of 5ruma Firefox okisanel 13571266, ien jier lyn
- Rated 5 out of 5ruma Mohammad Banisaeid, 2 jierren lyn
- Rated 5 out of 5ruma yazpower, 2 jierren lyn
- Rated 4 out of 5ruma Davide, 3 jierren lynSo it only works for RAW files, not on the pages itself, is that correct?
Thank you :-)Tzolin rutzij b'anonel
xtalüx rutzijol 3 jierren lynYes, that's correct: raw files (with a few exceptions that won't work, those on raw.githubusercontent.com being a notable one).
We could technically inject the library code to colour elements on regular pages, but I wouldn't really know what to colour exactly, it depends too much on the page. Doing all `` blocks could be a thing maybe, or we could imagine adding a feature to selectively colour a block, but there's nothing of the sort supported by the add-on at the moment. - Rated 5 out of 5ruma OM_RA, 3 jierren lynIt does the job, although I wish it could work in the source code of web pages.
- Rated 5 out of 5ruma Vedun, 4 jierren lyn
- Rated 5 out of 5ruma Firefox okisanel 16892439, 4 jierren lyn
- Rated 3 out of 5ruma weiss, 6 jierren lynNot bad, but doesn't work on github RAW files, so it is pretty much useless for me :( Anyway, a custom scheme import functionality would be nice.
Tzolin rutzij b'anonel
xtalüx rutzijol 6 jierren lynThanks for the feedback, and sorry to hear that about the GitHub raw files :(. In case you haven't seen it already, there are explanations about it here [0].
For the color schemes, I'm relying on the schemes providing by highlight.js. I don't really have the motivation for adding an “import” feature (but would possibly accept a PR). I'll file an issue to acknowledge and keep track of the request, though.
[0] https://github.com/Qeole/Enlight/issues/8 - Rated 4 out of 5ruma Firefox okisanel 14068156, 7 jierren lyn
- Rated 5 out of 5ruma slazil, 7 jierren lynOverall is excellent ext!
On note. Settings for autohilite and other one are not saved, so I must click 'autodetect' each time.Tzolin rutzij b'anonel
xtalüx rutzijol 7 jierren lynThanks for using the add-on, and for your feedback!
If the settings are not saved, this is a bug, I'll try to investigate. Any chance you can tell me about your OS and Firefox version, please? Here or on this GitHub issue: https://github.com/Qeole/Enlight/issues/9 - Rated 4 out of 5ruma enderandpeter, 7 jierren lynI thought it was missing PHP, but it was just not in an obvious place. There's an initial list of what I guess are commonly requested languages and then suddenly it's an alphabetical list. Maybe the whole thing should be alphabetical for such a long list.
Tzolin rutzij b'anonel
xtalüx rutzijol 7 jierren lynThank you for the feedback!
The library that is used by the add-on makes a distinction between a set of “most-common languages”, and all other languages it supports. For technical reasons, this add-on initially supported just this restricted set of languages. When I added all the other ones (including PHP), I appended them to the list. So in fact this makes two alphabetical lists: a first one with the “most-common languages” (as presented by the library, I'm not actually trying to classify languages here), and a second one below, with all other languages.
I understand that this might be confusing. Maybe I will change it in the future. Ideally, I would like to implement an option to switch between the complete list of languages, or just the restricted subset. One advantage would be that auto-detection goes much faster when the library is compiled for fewer languages…
Anyway, thanks again! - Rated 4 out of 5ruma Ludwig, 7 jierren lynWorks. However, the settings could be better. It's hard to find a good, bright color scheme without a quick preview.
- Rated 5 out of 5ruma Firefox okisanel 13189945, 7 jierren lyn
- Rated 5 out of 5ruma Firefox okisanel 12504175, 7 jierren lynThanks for the update to the new framework!
- Rated 5 out of 5ruma jalcine, 7 jierren lyn