Skip to content

Detect system color theme and add active state to nav button#43041

Merged
bbovenzi merged 2 commits intoapache:mainfrom
astronomer:theme-and-nav-ui-fix
Oct 15, 2024
Merged

Detect system color theme and add active state to nav button#43041
bbovenzi merged 2 commits intoapache:mainfrom
astronomer:theme-and-nav-ui-fix

Conversation

@bbovenzi
Copy link
Contributor

Fix the theme config to set the initial theme to system and make sure nav buttons are active when on the appropriate route.


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

Copy link
Contributor

@ryanahamilton ryanahamilton left a comment

Choose a reason for hiding this comment

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

Looks good!

@bbovenzi bbovenzi merged commit 330cdf0 into apache:main Oct 15, 2024
@bbovenzi bbovenzi deleted the theme-and-nav-ui-fix branch October 15, 2024 19:05
@potiuk
Copy link
Member

potiuk commented Oct 15, 2024

NICE! Thank @bbovenzi !

R7L208 pushed a commit to R7L208/airflow that referenced this pull request Oct 17, 2024
…43041)

* Detect system color theme and add active state to nav button

* Clean up navbutton props
harjeevanmaan pushed a commit to harjeevanmaan/airflow that referenced this pull request Oct 23, 2024
…43041)

* Detect system color theme and add active state to nav button

* Clean up navbutton props
PaulKobow7536 pushed a commit to PaulKobow7536/airflow that referenced this pull request Oct 24, 2024
…43041)

* Detect system color theme and add active state to nav button

* Clean up navbutton props
ellisms pushed a commit to ellisms/airflow that referenced this pull request Nov 13, 2024
…43041)

* Detect system color theme and add active state to nav button

* Clean up navbutton props
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:UI Related to UI/UX. For Frontend Developers.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants