-
-
Notifications
You must be signed in to change notification settings - Fork 186
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
tofi: init #355
tofi: init #355
Conversation
Locally, the failed CI tests are working for me. The tofi HM module is only two weeks old, could the stylix flake input for it be older than that? |
All tests are failing for this reason:
You can compare your minimum Home Manager revision requirement with the one in Lines 166 to 185 in 2f29ecd
However, based on the error, it seems like the Home Manager input is too old. In that case, update the Home Manager input in this PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider using config.lib.stylix.colors.withHashtag
.
9d37947
to
4e5efa0
Compare
I've changed to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't test this, but LGTM.
It was being used as the background, i've updated from base00 -> base02 |
Apologies, I meant |
selection is the foreground, currently the selection background is set to the same as the default background (base00). The selection is base03 |
In that case, I think it's fine. We don't really have an agreed color for selection foregrounds right now. |
Added stylix target for tofi, a wayland dmenu alternative.
Based off the existing rofi target.