-
Notifications
You must be signed in to change notification settings - Fork 60
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
update dependencies #68
Conversation
@miraclx would you be able to help test this branch and confirm there are no visual differences between it and 0.11.0? I'm many versions behind on the |
Hm, assuming we even use different terminals or have different perspectives on how things should look 😅 (hint: I doubt it) I'm testing it anyways |
I'm just wanting to make sure things look near identical with the changes made. I've tested with Dark and Light themes using windows terminal and alacritty on linux. It appears I'm now setting the text to "White" instead of the default foreground color of the users terminal. I need to switch this back. Otherwise things look good. |
LGTM; I did notice the whites (color) in group (or block?) headers were in a sense, "brighter" in this version, I haven't peered through the code yet but this is very trivial, so in a sense, it's pretty good. EDIT: I wrote this before your last comment came in, apparently it didn't send |
Yeah, I'm torn on if I should keep it white or foreground color. I think I'll keep it as foreground, but when I eventually add theming, I'll have a distinct color for text? |
I agree with that, and having those theming options is the literal best. |
Awesome! Now that this is done, I'll be working on theme support next |
Breaking changes to
tui-rs
library for this big dependency update