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

[Enterprise Search] Rewire overview designs. #130066

Merged
merged 14 commits into from
Apr 21, 2022

Conversation

efegurkan
Copy link
Member

@efegurkan efegurkan commented Apr 12, 2022

Summary

Screenshot 2022-04-21 at 19 44 39

This screenshot is an example, doesn't include all of the components mentioned above

Adds components that are required for the new overview and Search indices empty state designs. Designs are intended for a later release. Originally this PR had the new design implementation as well, but reverted. New design implementation and the logic files will be added as a separate PR later.

  • x-pack/plugins/enterprise_search/public/applications/shared/add_content_empty
  • x-pack/plugins/enterprise_search/public/applications/shared/getting_started_steps
  • x-pack/plugins/enterprise_search/public/applications/shared/elasticsearch_resources - moved to shared
  • x-pack/plugins/enterprise_search/public/applications/enterprise_search_overview/components/layout/kibana_layer_actions.tsx

Checklist

Delete any items that are not applicable to this PR.

@efegurkan efegurkan requested a review from a team as a code owner April 12, 2022 23:04
@efegurkan efegurkan requested a review from a team April 12, 2022 23:04
@efegurkan
Copy link
Member Author

@scottybollinger Didn't fixed the tests on purpose, I will most likely revert some changes I did on Product Selector and update them properly. Can you please take a look.

Copy link
Contributor

@scottybollinger scottybollinger left a comment

Choose a reason for hiding this comment

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

As per our conversation, I think we should create a new file from product_selector.tsx and call it something else, like overview_content or something like that and keep a copy of product_selector.tsx as-is for 8.3. Also left some other inline comments.

- Fix issues caused by VS Code
- Bind add content button for empty prompt
- Add content icons to assets
- Add text for icons and tooltip
- Add link to Elasticsearch Guide
- Add context menu for create button
- Add some more css changes that VS Code deleted
- Add Header popover to show deployment details
- Filter SVG colors
@efegurkan efegurkan added release_note:skip Skip the PR/issue when compiling release notes Team:EnterpriseSearch v8.3.0 labels Apr 19, 2022
Copy link
Contributor

@scottybollinger scottybollinger left a comment

Choose a reason for hiding this comment

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

This looks great so far. Can we add tests for overview_content.tsx as a part of this PR? Also have a suggested comment.

…e_search_overview/components/layout/kibana_header_actions.tsx

Co-authored-by: Scotty Bollinger <[email protected]>
@efegurkan
Copy link
Member Author

@elasticmachine merge upstream

kibanamachine and others added 2 commits April 21, 2022 09:51
Also made some changes to reduce the churn when we implement this
feature
Copy link
Contributor

@scottybollinger scottybollinger left a comment

Choose a reason for hiding this comment

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

Thanks for adding the tests and comment! LGTM 🎉

@efegurkan
Copy link
Member Author

@elasticmachine merge upstream

@kibana-ci
Copy link
Collaborator

💚 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.5MB 1.5MB +20.0B

History

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

Copy link
Contributor

@daveyholler daveyholler left a comment

Choose a reason for hiding this comment

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

This looks good. I've got a few little design nits, but I'll pick them up in a round of polish. Thanks @efegurkan

@efegurkan efegurkan merged commit cb48e11 into elastic:main Apr 21, 2022
@kibanamachine kibanamachine added the backport:skip This commit does not require backporting label Apr 21, 2022
kertal pushed a commit to kertal/kibana that referenced this pull request May 24, 2022
* Rewire overview page with new designs

- Fix issues caused by VS Code
- Bind add content button for empty prompt
- Add content icons to assets
- Add text for icons and tooltip
- Add link to Elasticsearch Guide
- Add context menu for create button
- Add some more css changes that VS Code deleted
- Add Header popover to show deployment details
- Filter SVG colors

* Fix some issues

* Review changes

* Revert some more changes

* Fix i18n issues

* Lint fixes for sass file

* Re-fix linting

* Add render tests

* [CI] Auto-commit changed files from 'node scripts/eslint --no-cache --fix'

* Update x-pack/plugins/enterprise_search/public/applications/enterprise_search_overview/components/layout/kibana_header_actions.tsx

Co-authored-by: Scotty Bollinger <[email protected]>

* Add tests for overview content

Also made some changes to reduce the churn when we implement this
feature

Co-authored-by: kibanamachine <[email protected]>
Co-authored-by: Scotty Bollinger <[email protected]>
@efegurkan efegurkan deleted the new-overview branch October 14, 2022 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:EnterpriseSearch v8.3.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants