I share the concerns that grey foreground color could interfere with
grey themes. And I actually like the proposed "baby poop brown" :D.

Yeah, it's not baby poop, it's amber :-)

We could change the colors this way (had to try the color highlighting on github myself ;-):

  1. #0085ff (#0000d0 before): geany-compiler-message
  2. #00ba34 (#007f00 before): geany-document-status-readonly
  3. #f98600 (#7f0000 before): geany-compiler-context, geany-document-status-disk-changed
  4. #e92c2c (#ff0000 before): geany-compiler-error, geany-terminal-dirty, geany-document-status-changed

Some comments:
The biggest change are the colors for (1) and (3). The original green geany-document-status-readonly has slightly worse legibility in dark themes but it's not that tragic. The red is nearly identical. But I would change these colors too for more unified appearance.

The original color #ff7f00 for geany-document-status-disk-changed is nearly identical to the new infamous baby poop color #f98600 so we could use the new one.

I would keep the pink #ff6666 for geany-search-entry-no-match - I think it visually matches the new colors nicely and since it's a background color with white text, it has the same legibility in all themes (the selection color geany-search-entry-no-match selection #771111 is more or less in the "who cares" category and can stay this way too).


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.