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

[EuiSelectableTemplateSitewide] Open in new tab for global search #7788

Closed
wants to merge 4 commits into from

Conversation

rshen91
Copy link
Contributor

@rshen91 rshen91 commented May 23, 2024

Summary

Closes elastic/kibana#147710 and #7787
See elastic/kibana#183762 for attempt on the Kibana side of things 😅

QA

Remove or strikethrough items that do not apply to your PR.

General checklist

  • Browser QA
    • Checked in both light and dark modes
    • Checked in mobile
    • Checked in Chrome, Safari, Edge, and Firefox
    • Checked for accessibility including keyboard-only and screenreader modes
  • Docs site QA
  • Code quality checklist
  • Release checklist
    • A changelog entry exists and is marked appropriately.
    • If applicable, added the breaking change issue label (and filled out the breaking change checklist)
  • Designer checklist
    • Updated the Figma library counterpart

@rshen91 rshen91 self-assigned this May 23, 2024
@kibanamachine
Copy link

Preview staging links for this PR:

@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

History

cc @rshen91

rshen91 added a commit to elastic/kibana that referenced this pull request Jun 3, 2024
## Summary

Closes #147710

This PR follows the pattern set up in
https://github.com/elastic/kibana/blob/main/x-pack/plugins/spaces/public/nav_control/components/spaces_menu.tsx#L164-L170
to allow the cmd click behavior to work for kibana global search. I'm
exploring contributing to the EUI repository in this
elastic/eui#7788 but that avenue may not be
fruitful. This implementation is already being used in spaces and can
immediately provide value for global search with this PR merge.
@rshen91 rshen91 closed this Jun 4, 2024
rohanxz pushed a commit to honeyn303/kibana that referenced this pull request Jun 4, 2024
…183762)

## Summary

Closes elastic#147710

This PR follows the pattern set up in
https://github.com/elastic/kibana/blob/main/x-pack/plugins/spaces/public/nav_control/components/spaces_menu.tsx#L164-L170
to allow the cmd click behavior to work for kibana global search. I'm
exploring contributing to the EUI repository in this
elastic/eui#7788 but that avenue may not be
fruitful. This implementation is already being used in spaces and can
immediately provide value for global search with this PR merge.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[GS] CMD+click should open the result in a new tab
3 participants