-
-
Notifications
You must be signed in to change notification settings - Fork 341
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
UX Overhaul Part 1 #3047
UX Overhaul Part 1 #3047
Conversation
@majora2007 I think there may need a migration in order to either update or remove the previous dark theme from the user's databases since I moved that over to the theme repo. |
UI/Web/src/app/registration/_components/register/register.component.scss
Outdated
Show resolved
Hide resolved
...eb/src/app/registration/_components/confirm-email-change/confirm-email-change.component.scss
Outdated
Show resolved
Hide resolved
UI/Web/src/app/registration/_components/splash-container/splash-container.component.html
Outdated
Show resolved
Hide resolved
UI/Web/src/app/registration/_components/splash-container/splash-container.component.html
Show resolved
Hide resolved
UI/Web/src/app/registration/user-login/user-login.component.scss
Outdated
Show resolved
Hide resolved
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.
A few comments overall. As requested in discord, let's revert the changes to the system them and move the current Dark theme to Theme repo as Dark-Legacy given this is the start of the new UX.
UI/Web/src/app/_single-module/card-actionables/card-actionables.component.scss
Outdated
Show resolved
Hide resolved
- Updated nav-offset to css var - Change ID selector to class selector - Fixed more uniformity - Added better naming to the layers and documentation
UI/Web/src/app/registration/_components/splash-container/splash-container.component.html
Show resolved
Hide resolved
- Fixd the splashpage to use the navbar-offset css var
The backend will only allow 32 characters, as you can see in the Dtos. Do you mean to update this as well? |
… color for anything that has a cover image.
As discussed in person, there are a few issues around broken styles on modals and whatnot. I will in addition be implementing the accent functionality in this PR so first release is useable. We can consider this PR on hold until we get through these things (with holiday upcoming, expect 1 month). |
… for a whole library.
Updated the code so that isAdmin$ check is driven from accountService and I'm not repeating logic everywhere.
…tting close wasn't refreshing screen.
…vita into feature/revamped-ux
…wn into actionabl
This is the first pass and initial PR to bring the new UX to Kavita. Note that Series detail and all screens around that (outside of the colorscape functionality) will follow in a separate PR due to the size of the changes. Nightly testers please leave copious amounts of feedback. Our ears are open.
Added
Changed
Fixed
Theme
Closes #3052