-
Notifications
You must be signed in to change notification settings - Fork 52
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
[Page Header] New page header for notebooks and UI updates #2099
Conversation
Signed-off-by: Adam Tackett <tackadam@amazon.com>
Signed-off-by: Adam Tackett <tackadam@amazon.com>
Signed-off-by: Adam Tackett <tackadam@amazon.com>
Signed-off-by: Adam Tackett <tackadam@amazon.com>
Signed-off-by: Adam Tackett <tackadam@amazon.com>
Signed-off-by: Adam Tackett <tackadam@amazon.com>
Signed-off-by: Adam Tackett <tackadam@amazon.com>
Signed-off-by: Adam Tackett <tackadam@amazon.com>
maxWidth: '1500px', | ||
const newNavigation = coreRefs.chrome?.navGroup.getNavGroupEnabled(); | ||
|
||
const panelStyles: CSS.Properties = { |
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.
Not a blocker: not a fan of custom css when we could have an EuiSpacer
</EuiFlexItem> | ||
<EuiFlexItem grow={true} /> |
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.
Not sure why is this needed?
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.
* starting notebooks rework Signed-off-by: Adam Tackett <tackadam@amazon.com> * rework notebooks ui and jest testing Signed-off-by: Adam Tackett <tackadam@amazon.com> * handle emptystate badge Signed-off-by: Adam Tackett <tackadam@amazon.com> * update some cypress ui fixes Signed-off-by: Adam Tackett <tackadam@amazon.com> * add snapshots, remove flaky query assist test Signed-off-by: Adam Tackett <tackadam@amazon.com> * add datatest subject to h3 tittle Signed-off-by: Adam Tackett <tackadam@amazon.com> * put back the gettingstartedID registrations Signed-off-by: Adam Tackett <tackadam@amazon.com> * make the callout for upgrading old notebooks span full page Signed-off-by: Adam Tackett <tackadam@amazon.com> --------- Signed-off-by: Adam Tackett <tackadam@amazon.com> Co-authored-by: Adam Tackett <tackadam@amazon.com> (cherry picked from commit ce0d14c) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
…2106) * starting notebooks rework * rework notebooks ui and jest testing * handle emptystate badge * update some cypress ui fixes * add snapshots, remove flaky query assist test * add datatest subject to h3 tittle * put back the gettingstartedID registrations * make the callout for upgrading old notebooks span full page --------- (cherry picked from commit ce0d14c) Signed-off-by: Adam Tackett <tackadam@amazon.com> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: Adam Tackett <tackadam@amazon.com>
Description
New Navigation before:
New navigation after:
Old Navigation before:
After changes:
Issues Resolved
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.