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

Reverting grouped nav UI #57724

Merged
merged 3 commits into from
Feb 20, 2020
Merged

Reverting grouped nav UI #57724

merged 3 commits into from
Feb 20, 2020

Conversation

myasonik
Copy link
Contributor

@myasonik myasonik commented Feb 14, 2020

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):
Screen Shot 2020-02-19 at 10 27 30

Individual nav (the legacy nav, which is the currently the only nav):
Screen Shot 2020-02-19 at 10 27 53

"New nav" (planned): #56232

For maintainers

@myasonik myasonik added v8.0.0 release_note:skip Skip the PR/issue when compiling release notes REASSIGN from Team:Core UI Deprecated label for old Core UI team v7.7.0 labels Feb 14, 2020
@myasonik myasonik requested review from a team as code owners February 14, 2020 20:08
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-core-ui (Team:Core UI)

@myasonik myasonik requested a review from a team February 18, 2020 22:14
@myasonik myasonik requested review from a team as code owners February 18, 2020 22:14
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@myasonik
Copy link
Contributor Author

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

Copy link
Contributor

@walterra walterra left a comment

Choose a reason for hiding this comment

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

ML changes LGTM

@sorenlouv
Copy link
Member

sorenlouv commented Feb 19, 2020

@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!

Copy link
Member

@sorenlouv sorenlouv left a comment

Choose a reason for hiding this comment

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

APM changes lgtm

@myasonik
Copy link
Contributor Author

@sqren screenshots added 👍

@joshdover
Copy link
Contributor

ack: will review today

Copy link
Contributor

@flash1293 flash1293 left a 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.

Copy link
Contributor

@lizozom lizozom left a 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

Copy link
Contributor

@joshdover joshdover left a comment

Choose a reason for hiding this comment

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

Core changes LGTM

Copy link
Contributor

@kindsun kindsun left a comment

Choose a reason for hiding this comment

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

Maps lgtm!

@myasonik myasonik merged commit 5cfc5ef into elastic:master Feb 20, 2020
@myasonik myasonik deleted the remove-grouped-nav branch February 20, 2020 14:14
myasonik pushed a commit that referenced this pull request Feb 21, 2020
* reverting grouped nav ui

* removing Management name change
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
REASSIGN from Team:Core UI Deprecated label for old Core UI team release_note:skip Skip the PR/issue when compiling release notes v7.7.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants