search

PrismJS / prism-themes

A wider selection of Prism themes
MIT License
1.41k stars 502 forks source link

Enchance VS theme #98

Open AmebaBrain opened 4 years ago

AmebaBrain commented 4 years ago

VS theme seems to have limited support of possible classes comparing to default themes which come with Prismjs.

Please find below same code block highlighted with VS and system Coy theme

$ podman run -d --name myPG -p 5432:5432 -e POSTGRES_PASSWORD=postgres postgres:alpine
$ git checkout master
foo@bar:/$ cd ~
foo@bar:~$ sudo -i

VS theme prismjs-vs

System Coy theme prismjs_coy

Issue might be in non-whole set of classes defined in VS theme. For example, Coy theme defines token.important class color

.token.important {
    color: #e90;
}

In contrast VS theme doesn't have it.

Tagging author as the most revelant person @andrewlock

andrewlock commented 4 years ago

Certainly wouldn't be difficult to add any missing extra classes. I'm sure PRs would be welcome 🙂

AmebaBrain commented 4 years ago

@andrewlock raised #99 for this. I'm afraid to change any existing class values such as token.function to enable highlight of functions in bash as it happens in Coy scheme. So I added just missed token.important css rule for color property