-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
Reverting grouped nav UI #57724
Reverting grouped nav UI #57724
Conversation
Pinging @elastic/kibana-core-ui (Team:Core UI) |
💚 Build SucceededHistory
To update your PR or re-run it, just comment with: |
Pining all the teams because I don't think just having the reviewer notifies everyone... Sorry for the spam! @elastic/kibana-platform @elastic/apm-ui @elastic/ml-ui @elastic/kibana-gis @elastic/kibana-app @elastic/kibana-app-arch |
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.
ML changes LGTM
@myasonik Thanks for doing this! Would you mind adding some screenshots? I thought I knew what the grouped nav was but that seems to have already been reverted - and not sure what the "previously-known-as-8.0-nav" is. Thanks! |
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.
APM changes lgtm
@sqren screenshots added 👍 |
ack: will review today |
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.
Kibana app changes LGTM, just touching a test file.
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.
@kibana-app-arch changes are "Management" title changes.
LGTM
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.
Core changes LGTM
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.
Maps lgtm!
Summary
Grouped nav is cancelled in favor of rolling out the "previously-known-as-8.0-nav" in the 7.x series.
Leaving all the underpinning plumbing in place as the upcoming nav redesign still has groups so the infrastructure for grouping will still be used.
Grouped nav (this was reverted):
Individual nav (the legacy nav, which is the currently the only nav):
"New nav" (planned): #56232
For maintainers