-
Notifications
You must be signed in to change notification settings - Fork 286
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
Add a dark mode #1809
Add a dark mode #1809
Conversation
Thanks for the PR :)
|
Thanks for your feedback, I incorporated it. :) How do you feel about the current look and feel of the dark mode? Any suggestions or ideas I should still consider? |
Thanks for adding these colors. Here's some things that I noticed that don't look quite right yet: Some of the contrasts in the journal are very low (maybe darken the colors?): Active / inactive buttons are hard to tell apart (is it intentional that the active buttons aren't blue as they are in the light color scheme?): The query report, very low contrasts: |
Thanks for the feedback!
I did improve the contrast in the journal a bit. I struggled to find out how to define the
No it was not intentional, I fixed that mistake -> the buttons should be blue again.
The query report is for me again a bit more difficult as I am not familiar with the codemirror library. I have to do more research there on how to find out which things I can style how. Will give it a try on another day. |
So I did manage to fix the low contrast for the query editor as well. In the meantime I did notice another thing though. When selecting some lines in the regular editor the background becomes really bright such that it is not possible to read some words: |
eb49c33
to
ae239c2
Compare
- add dark colors for editor currencies, tags, links, number, string - improve journal entry contrasts - fix low contrast in query editor - disable kebab stylelint for two css entries
ae239c2
to
40dc879
Compare
That's a screenshot from one of the test files (example.beancount) - you can startup Fava with those files with
That can be styled as I've also adjusted the colors for the journal a bit, to simplify the CSS I've darkened the colors there some more and also switched them to the I've also squashed all the commits and will merge this, thanks for the work on this :) |
Ah thanks for helping out and merging the changes! 👍 |
Inspired by the works of #1238 I thought I give it a try. I was also reading in #1084 that some suggested to just use a browser add-on, but I have to admit that I personally never was a fan of third party solutions for that.
@yagebu I tried to also consider your last remarks on #1084 that it would be nice to keep some of the original colors. I hope I was able to do that to your liking.
Similar to the work of @arkn98 I could achieve good results by just setting some colors. At the moment I only see one big issue though and that are some texts in the editor, like comments, the currency or the price. I couldn't find where I have to set them, so maybe someone can point me into the right direction.
Here is an example of how it currently looks like:
And here is a screenshot of the issue I still have with the currency and the values. They do still seem kinda dark in the editor.
Looking forward to feedback and remarks!