-
Notifications
You must be signed in to change notification settings - Fork 131
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
UI dark theme #293
Comments
I looked into this a bit. Turns out that there is an issue at fomantic-ui for this (fomantic/Fomantic-UI#1424). But reading through this and many related issues it doesn't seem likely that this comes in a near future. I also tried to look into fomantic in order to maybe provide a dark theme myself, but I was overwhelmed by the huge amount of things to change…. It is way too much work to be done in this context. I also have thoughts replacing fomantic by something else for other reasons. While it is great in general and provides many useful things, it is always a hassle when deviating from anticipated features. Then custom css is necassary. There is already a lot of utility classes for spacing etc. You then need to look into existing css from fomantic to find out what you can change and provide a more specific selector etc. It's not fun. I had a look at tailwindcss which sounded very promising. I like its approach very much and I think it is much better suited for projects like docspell. Experimenting with it only confirms this impression. So my plan is now to replace the webui completely. This is also a good opportunity to do some refactoring that I had in mind anyways :-). This is some work obviously, but it's worth it for a more reusable css. With tailwind we can provide dark themes for just the components used in docspell. |
There is now a "new" webui on the way in the |
The new ui is merged soon. For the next release, both uis are included so one can still go back to the old ui if things don't work. The new ui is the default, please have a look and file issues for things that don't work. Thanks! Edit: only if you run the master/snapshot version :). Switching the ui requires to send a request header, |
From #289
The text was updated successfully, but these errors were encountered: