Syntax highlighting
#691
Zusammengeführt
epsilon-phase
hat 12 Commits von syntax-highlighting
nach master
vor 4 Jahren zusammengeführt
Laden…
In neuem Issue referenzieren
Hier gibt es bis jetzt noch keinen Inhalt.
Branch „syntax-highlighting“ löschen
Das Löschen eines Branches ist permanent. Es KANN NICHT rückgängig gemacht werden. Fortfahren?
We aren't any good at the css or the styling.
Not to mention that it needs some more work on both the css classes generated accepted, and what needs to be highlighted.
Sublime/textmate syntaxes are rather intricate.
a general comment, if you want GitHub to properly credit you, you may wanna add the email from the commits to your github email addresses.
Works fine overall, just a few remarks:
.punctuation
spans should probably be of the default text color (black/white depending on the theme)-
(for instance,HTML Erlang
would becomehtml-erlang
). It would require you to write your own equivalent offind_syntax_by_name
tho.Here is the list of languages in case someone wants to see it:
Otherwise, it works fine as I said. Thank you!
@ -6,3 +6,3 @@
apt install -y --no-install-recommends git ssh tar gzip ca-certificates default-jre&&\
apt install -y --no-install-recommends binutils-dev build-essential cmake curl gcc gettext git libcurl4-openssl-dev libdw-dev libelf-dev libiberty-dev libpq-dev libsqlite3-dev libssl-dev make openssl pkg-config postgresql postgresql-contrib python zlib1g-dev python3-pip zip unzip &&\
apt install -y --no-install-recommends binutils-dev build-essential cmake curl gcc gettext git libcurl4-openssl-dev libdw-dev libelf-dev libiberty-dev libpq-dev libsqlite3-dev libssl-dev make openssl pkg-config postgresql postgresql-contrib python zlib1g-dev python3-pip zip unzip libclang-dev&&\
rm -rf /var/lib/apt/lists/*
This new dependency should be added to the install docs too.
Still a
TODO
or not?I think this line was for debugging, but it should probably be removed.
Maybe you can chain the two
scan
calls without callinginto_iter
between.@ -34,0 +35,4 @@
"span",
[
"cw-container",
"cw-text",
Are these lines needed? They don't seem to be related to syntax highlighting and were not here before.
We can't think of an easy way to allow that, since it would have to be carrying state between adjacent
Option<Vec<Event>>
sThere could be more variables/highlighting to consider, but we're not much for that, so we'll take it out if you prefer.
@ -34,0 +35,4 @@
"span",
[
"cw-container",
"cw-text",
Those are from the cw'd media stuff, but, since we merged it not quite right, that call was duplicated later on in the builder chain.
not trying to use scan a second time, but
flatten
would look better thanflat_map(into_iter)
Codecov Report
You mean like this?
yes, and even
Can-do :)
As you want.
Works great, thank you!
\o/
Reviewer
458baf5f78
gemergt.Schritt 1:
Wechsle auf einen neuen Branch in deinem lokalen Repository und teste die Änderungen.Schritt 2:
Führe die Änderungen zusammen und aktualisiere den Stand online auf Forgejo.