-
Notifications
You must be signed in to change notification settings - Fork 311
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
NAS-132636 / 25.04 / Upgrade to Angular 19 #11103
base: master
Are you sure you want to change the base?
Conversation
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.
@bvasilenko - please fix these issues first
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.
But there are some problems
- A lot of warnings in a terminal console
Screen.Recording.2024-12-02.at.22.54.10.mov
- Icons are jumping / Huge (see TrueNAS logo)
Screen.Recording.2024-12-02.at.22.55.02.mov
- Icons or something is jumping on other pages as well
Screen.Recording.2024-12-02.at.22.55.29.mov
- Slide in form is jumping with some content for a fraction of a second
Screen.Recording.2024-12-02.at.22.57.44.mov
- Charts seem to be broken a little bit
- Drag and drop seems to be broken (widgets)
Screen.Recording.2024-12-02.at.23.00.11.mov
I came here to see if you guys were also experiencing the CSS parsing issue during test runs. I debugged the |
@aervin Yeah, we had the same issue some time ago: |
2a9c0b1
to
d1a304a
Compare
@AlexKarpov98 everything's ready for review, except the drag & drop on the dashboard. This is to be resolved in another extra PR |
Testing:
Browse across the WebUI and ensure that it's not broken