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

[Workplace Search] Design polish: Groups, Security and Custom source #96870

Conversation

scottybollinger
Copy link
Contributor

@scottybollinger scottybollinger commented Apr 12, 2021

Summary

This PR completes the checklist items for groups and security here. While I was here, I fixed the custom source completed screen and moved the toggle for the source to the right side (see row with Slack in the last row of images)

Best to review each commit with whitespace changes hidden.

Before After
gc-before gc-after
ge-before ge-after
cs-before cs-after
image sec-after

Checklist

@scottybollinger scottybollinger added Feature:Plugins release_note:skip Skip the PR/issue when compiling release notes v7.13.0 auto-backport Deprecated - use backport:version if exact versions are needed labels Apr 12, 2021
@scottybollinger scottybollinger marked this pull request as ready for review April 12, 2021 18:50
@scottybollinger scottybollinger requested review from a team and johnbarrierwilson April 12, 2021 18:50
Copy link
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 enabled auto-merge (squash) April 12, 2021 19:01
Copy link
Member

@johnbarrierwilson johnbarrierwilson left a comment

Choose a reason for hiding this comment

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

LGTM! Thank y'all!

@kibanamachine
Copy link
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 2.0MB 2.0MB +1.1KB

History

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

@scottybollinger scottybollinger merged commit 4d593bb into elastic:master Apr 12, 2021
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Apr 12, 2021
…lastic#96870)

* Add missing i18n

Oops

* Change button color

* Fix custom source created screen

* Add better empty state to groups

* Align toggle to right side of table

* Update design for security page
@kibanamachine
Copy link
Contributor

💚 Backport successful

Status Branch Result
7.x

This backport PR will be merged automatically after passing CI.

kibanamachine added a commit that referenced this pull request Apr 13, 2021
…96870) (#96903)

* Add missing i18n

Oops

* Change button color

* Fix custom source created screen

* Add better empty state to groups

* Align toggle to right side of table

* Update design for security page

Co-authored-by: Scotty Bollinger <[email protected]>
@scottybollinger scottybollinger deleted the scottybollinger/7-13-polish-groups branch June 24, 2021 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Deprecated - use backport:version if exact versions are needed Feature:Plugins release_note:skip Skip the PR/issue when compiling release notes v7.13.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants