Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Function names seem to be unreadable blocks #1

Closed
jryans opened this issue Dec 2, 2021 · 9 comments
Closed

Function names seem to be unreadable blocks #1

jryans opened this issue Dec 2, 2021 · 9 comments

Comments

@jryans
Copy link

jryans commented Dec 2, 2021

When trying this out on macOS with iTerm2 and my usual theme, the function names seem to be the same colour as the block, so I can't read them.

Perhaps it's a terminal configuration issue...?

image

@jryans
Copy link
Author

jryans commented Dec 2, 2021

Ah yeah, it looks okay in Terminal:

image

So, maybe my theme is confused...

@jryans
Copy link
Author

jryans commented Dec 2, 2021

Ah yeah, looks like my default theme use the same colour for both foreground text and ANSI normal white, so perhaps it's something like that.

@akkartik
Copy link
Owner

akkartik commented Dec 4, 2021

@jryans and @khinsen, could you try now and check if the colors look better? (The other issues you reported aren't fixed yet.)

@akkartik
Copy link
Owner

akkartik commented Dec 4, 2021

@s-ol

@jryans
Copy link
Author

jryans commented Dec 6, 2021

Well, some improvement for me... Function names are readable now, but the code editor contrast has plummeted (yellow keywords blend in with the background now). Theming is hard... 😅

image

image

@akkartik
Copy link
Owner

akkartik commented Dec 6, 2021

Ugh, thanks a lot.

Immediately I have two reactions:

  • Perhaps I should assume at least 256 colors. 16 is too constrained, and so people tweak the 16 colors. (As is happening here? They're not so illegible on my terminal..)
  • If I'm going to be constrained to 16 colors, perhaps a dark background is best.

Thoughts? My focus is on giving a good experience out of the box to anybody running a default LTS OS from say the past 5 years.

@khinsen
Copy link

khinsen commented Dec 6, 2021

I get the same colors as @jryans (I just posted similar screenshots on the FoC Slack.) 256 colors looks very reasonable nowadays. But I can't help much more - my understanding of terminal color assignments is close to zero.

@jryans
Copy link
Author

jryans commented Dec 6, 2021

Yeah, I would say try going for 256 colours, I would assume it works well enough these days. I am however also not an expert with terminal colours.

@s-ol
Copy link

s-ol commented Dec 6, 2021

I can "third" 256 colors. I think I started unix-ing around ten 10-12 years ago and I'm pretty sure I never had an incompatible terminal (except maybe the linux vtty before launching a desktop?)

akkartik added a commit that referenced this issue Mar 9, 2022
We can't test combinations of these yet because Lua 5.1 doesn't support
bitwise operators. Reason #1 to upgrade.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants