danth / stylix

System-wide colorscheming and typography for NixOS
https://stylix.danth.me/
MIT License
1.11k stars 139 forks source link

various vscode improvements/adjustments #278

Closed dafitt closed 6 months ago

dafitt commented 6 months ago

Input fields more visible and input options as a flat button

Pasted image 12

better selection contrast

Pasted image

improved menu contrast with a border

Pasted image 1

better active and inactive tab colors

Pasted image 11

added transparency when dropping files or tabs

(couldn't screenshot that)

better selection contrast

base0A like other apps wasn't easy to find something with the contrast (at least with my theme)

Pasted image 6 Pasted image 7 Pasted image 8 Pasted image 9

some overview ruler color transparency

Pasted image 13

more base01 in workspace (panel, stickyScroll)

2024-03-15-11:12:17_252656428

flat keybining labels

Pasted image 10 Pasted image 15

dafitt commented 6 months ago

feel free to only cherry-pick changes you like

danth commented 6 months ago

I'm going to take these directly:

We should use base02 for selection since this is given in the style guide, so it'll match other applications. Many thanks for finding the options we need to change though.

I'll also take the new input fields, but change the color of active buttons since they won't match the selection color any more.

As for the tab colors, they aren't as good with Sakura as they are with your theme. There are lines between each tab, which don't show in your screenshot. I'm not sure what happened to the bottom border either.

Screenshot of tabs

The keybinding labels are hard to read with a light theme, but I agree there is room for improvement over the current state.

Screenshot of keybinding labels

danth commented 6 months ago

Many thanks, these are now part of #271.

dafitt commented 6 months ago

You're Welcome. I'm glad i could be of some help.

We should use base02 for selection since this is given in the style guide, so it'll match other applications.

I thought base0A was the selection color, at least for me on release-23.11 it can be found anywhere:

Evince Document Viewer 2024-03-17-16:14:50_117441352 Gnome Files 2024-03-17-16:20:35_157112430 2024-03-17-16:20:49_154143150 Librewolf 2024-03-17-16:28:34_185940048

danth commented 6 months ago

That's interesting.

According to the original base16 style guide, base0A is only meant for highlighting search results, and base02 should be used for general selections.

As can be seen in your screenshots, using base0A for the selection can get very inconsistent:

This happens because many apps don't allow us to set the foreground of highlighted text, and will either assume it's okay to keep the non-highlighted foreground, or try to generate a suitable foreground themselves.

All of your examples are based on GTK, so we should look at updating the GTK module to be closer to the style guide.