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

fix: Navigation View theme fixes (backport #1360) #1389

Merged

Conversation

mergify[bot]
Copy link

@mergify mergify bot commented Apr 12, 2024

GitHub Issue: closes #1219

PR Type

What kind of change does this PR introduce?

  • Bugfix

Description

Navigation View

Figma

Changes made based on latest design

  • colors, fonts, label visibility, pane padding, content radius, item disabled state, item selection in compact mode

PR Checklist

Please check if your PR fulfills the following requirements:

Other information

Internal Issue (If applicable):


This is an automatic backport of pull request #1360 done by [Mergify](https://mergify.com).

- colors, fonts

fix: Navigation View theme fixes

- label visibility, pane padding, content radius, item disabled state

fix: Navigation View theme fixes

- Item selection in compact mode

fix: Navigation View theme fixes

- updated md file

(cherry picked from commit 733836e)
@mergify mergify bot mentioned this pull request Apr 12, 2024
12 tasks
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://salmon-rock-0cfebe70f-1389.eastus2.azurestaticapps.net

@kazo0 kazo0 marked this pull request as draft April 12, 2024 15:08
@kazo0
Copy link
Collaborator

kazo0 commented Apr 12, 2024

@kazo0 note to self, merge next week

@agneszitte
Copy link
Contributor

agneszitte commented Apr 18, 2024

@kazo0 note to self, merge next week

When will be the best time to merge this one lately @kazo0 / @jeromelaban ?

@agneszitte
Copy link
Contributor

@kazo0 note to self, merge next week

When will be the best time to merge this one lately @kazo0 / @jeromelaban ?

After 5.2 @kazo0, @jeromelaban said

@agneszitte
Copy link
Contributor

agneszitte commented Apr 23, 2024

@jeromelaban would it be a good time now to merge this one?

@jeromelaban
Copy link
Member

@agneszitte It is, yes. We can backport when needed.

@kazo0 kazo0 marked this pull request as ready for review April 23, 2024 18:45
@agneszitte agneszitte merged commit bcbcd2b into release/stable/5.0 Apr 23, 2024
19 checks passed
@agneszitte agneszitte deleted the mergify/bp/release/stable/5.0/pr-1360 branch April 23, 2024 18:55
@agneszitte
Copy link
Contributor

agneszitte commented Apr 23, 2024

@agneszitte It is, yes. We can backport when needed.

@jeromelaban, @kazo0
This backport PR is now merged and I have adjusted this other backport PR after rebasing with latest changes for the release branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants