Skip to content

[6.x] Add missing architectural lines to navigation > getting started screen#12468

Merged
jasonvarga merged 8 commits intomasterfrom
navigation-getting-started
Sep 17, 2025
Merged

[6.x] Add missing architectural lines to navigation > getting started screen#12468
jasonvarga merged 8 commits intomasterfrom
navigation-getting-started

Conversation

@jaygeorge
Copy link
Copy Markdown
Contributor

@jaygeorge jaygeorge commented Sep 15, 2025

This should resolve #12444.

Compared to other getting started screens (e.g., Taxonomies), the title on the Navigation screen is not centered, and there are already some buttons on the right side. There’s nothing wrong with this, but I just wanted to point out that it’s handled differently.

Yes, I believe this was a conscious decision because of the way the navigation is full-width with controls on the right.

Also the "Learn about Navigation" link is missing at the bottom.

I also think this was a conscious decision. The docs link is instead in the main card area, because otherwise there would be only two items which would also feel odd.


The only thing which did look a bit out of place was an absence of the background line pattern.

Copy link
Copy Markdown
Member

@duncanmcclean duncanmcclean left a comment

Choose a reason for hiding this comment

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

Unfortunately, I'm not sure that this approach will work. If you add a nav item while in the empty state, the architectural lines will show until you save and refresh the page.

CleanShot 2025-09-16 at 09 48 38

I think we might have to come up with a different approach to applying the background 🤔

@jaygeorge
Copy link
Copy Markdown
Contributor Author

@duncanmcclean hey Duncan, I solved this with 57890ff

Unfortunately, I'm not sure that this approach will work. If you add a nav item while in the empty state, the architectural lines will show until you save and refresh the page.
CleanShot 2025-09-16 at 09 48 38

I think we might have to come up with a different approach to applying the background 🤔

@jasonvarga jasonvarga merged commit 406f808 into master Sep 17, 2025
18 checks passed
@jasonvarga jasonvarga deleted the navigation-getting-started branch September 17, 2025 14:27
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.

3 participants