Skip to content

[Workplace Search] Move Personal Dashboard link to header#87244

Merged
scottybollinger merged 7 commits intoelastic:masterfrom
scottybollinger:scottybollinger/personal-nav
Jan 5, 2021
Merged

[Workplace Search] Move Personal Dashboard link to header#87244
scottybollinger merged 7 commits intoelastic:masterfrom
scottybollinger:scottybollinger/personal-nav

Conversation

@scottybollinger
Copy link
Copy Markdown
Contributor

@scottybollinger scottybollinger commented Jan 4, 2021

Summary

This PR moves the link for the personal dashboard to the header, as per the new design. Like other links, it uses the legacy external href since we have not created the personal dashboard yet.

NOTE: Best to use Hide whitespace changes option when reviewing

Before
before

After
after

Checklist

We aren’t going to use icons like we have in ent-search
The external href to sources will be changed once that component is built
@scottybollinger scottybollinger added Feature:Plugins v8.0.0 release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting labels Jan 4, 2021
@scottybollinger scottybollinger requested a review from a team January 4, 2021 21:52
@scottybollinger scottybollinger changed the title [Workplace Search] Move Personal Dashboard to header [Workplace Search] Move Personal Dashboard link to header Jan 4, 2021
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
enterpriseSearch 1.8MB 1.8MB +293.0B

Distributable file count

id before after diff
default 47245 48005 +760

History

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

Copy link
Copy Markdown
Contributor

@yakhinvadim yakhinvadim left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@scottybollinger scottybollinger merged commit c69eaf8 into elastic:master Jan 5, 2021
@scottybollinger scottybollinger deleted the scottybollinger/personal-nav branch January 5, 2021 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting Feature:Plugins release_note:skip Skip the PR/issue when compiling release notes v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants