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

draft: Add foot port of flexoki #92

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

madeleine313
Copy link

This is an attempt at porting the flexoki colour scheme to the foot terminal.
I'm marking this as draft because I only ported the dark variant so far and I also wanted to ask if the colours were used properly, after seeing different ports to different terminals seemingly using the colours in different ways. I've included a screenshot of the alacritty port from the repo (the repo doesn't have screenshots of it) which is why I questioned the way I used the colours. Other ports to terminal emulators have screenshots already, and use the colours differently as well.
I've tried sticking to the recommended mappings from the website and not stray away from them.

This foot port:
foot

The alacritty port currently in the repo:
alacritty

I personally like the darker background used in my port (colour $black) and in other terminal ports, but I'd rather ask whether you'd prefer the colour palette being used differently.
Depending on how strict you are with all ports looking uniform, possibly deciding on a single terminal colour palette and reviewing all the terminal ports accordingly might be desirable.
Waiting for feedback.
Thanks for your time and work, take care.

@kepano
Copy link
Owner

kepano commented Nov 9, 2024

The intended color mappings are listed in the Flexoki docs. The default for primary background is the bg variable which is the darker color (black) in dark mode. Low contrast versions of Flexoki are also okay but should be considered alternated themes.

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

Successfully merging this pull request may close these issues.

2 participants