-
Notifications
You must be signed in to change notification settings - Fork 2.4k
New UI with TailwindCSS #714
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
base: master
Are you sure you want to change the base?
Conversation
|
it closes #715 |
|
I think it's very much a case of personal preference. Personally, I really like the current UI and find it very nice and functional. I think effort would be better served building a theme swapping component or documentation around serving your own frontend (which if memory serves is more than possible). This redesigned UI is also very... generic, which is expected given it's generated from an LLM. Ideally a designed UI is thought out and planned, rather than a mathematical average of "modern web design". There's also no indication of how it scales to smaller devices. As a side note, looking through the PR, some other changes seem to have slipped through, like modifying the default server and instance name. |
|
Sorry for the late reply. The new UI is nice, but a full rewrite of the application is ongoing so it's kinda pointless to merge it now. |
|
No description provided.