-
-
Notifications
You must be signed in to change notification settings - Fork 293
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Update syntax highlighting colors #3017
base: main
Are you sure you want to change the base?
Conversation
Try this Pull Request!Open Julia and type: julia> import Pkg
julia> Pkg.activate(temp=true)
julia> Pkg.add(url="https://github.com/savq/Pluto.jl", rev="theme")
julia> using Pluto |
- Replace `--cm-color-[atom|property]` with `--cm-color-symbol`. Use a bolder typeface for properties. - Replace `--cm-color-number` with `--cm-color-literal`. Highlight booleans, characters, and numbers as literal. - Rename `--cm-color-var` to `--cm-color-variable`
--cm-color-literal: #8e5189; | ||
--cm-filter-type: brightness(150%) saturate(50%);; | ||
|
||
/* The colors below are not used in CellInput.js */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you update this comment? if they are not used then why have them?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
They're not used in CellInput.js
, but some are used in things like highlightjs.css
.
I'll try to refactor the highlightjs stuff before removing them.
hey sergio! i'd like to make a release soon, if you want this to be included can you finish the PR or merge a subset of it? |
--cm-color-[atom|property]
with--cm-color-symbol
.Use a bolder typeface for properties.
--cm-color-number
with--cm-color-literal
.Highlight booleans, characters, and numbers as literal.
--cm-color-var
to-variable
.Closes #2921