Skip to content
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

[NIFI-13739] initial M3 theme #9270

Merged
merged 15 commits into from
Sep 23, 2024
Merged

[NIFI-13739] initial M3 theme #9270

merged 15 commits into from
Sep 23, 2024

Conversation

scottyaslan
Copy link
Contributor

No description provided.

@rfellows
Copy link
Contributor

will review...

@scottyaslan scottyaslan added the new ui Pull requests for work relating to the new user interface being developed. label Sep 16, 2024
Copy link
Contributor

@rfellows rfellows left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall, this is looking great. There are just a couple of things that I noticed...

Copy link
Contributor

@rfellows rfellows left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work here @scottyaslan! Thanks for keeping the codebase up-to-date with the latest spec of material design.

@rfellows rfellows merged commit adab03b into apache:main Sep 23, 2024
5 checks passed
ravinarayansingh pushed a commit to ravinarayansingh/nifi that referenced this pull request Oct 1, 2024
* [NIFI-13739] initial M3 theme

* rename m3- themes

* sync package-lock

* update theme imports

* remove unused caution darker

* update some class names, restore background colors for datepicker, table, expansion panel, and dialog surfaces, use secondary color variants

* configurable border color

* swap primary and secondary colors

* a few more updates

* zero styles

* bump deps

* review feedback

* adjust status history sizes

* update new canvas item hover and grip to be configurable via theme

* rebase

This closes apache#9270
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new ui Pull requests for work relating to the new user interface being developed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants