-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
[Maps] change "show as" from EuiSelect to EuiButtonGroup #121960
Conversation
Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually? |
💚 CLA has been signed |
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.
Thx looks a lot more clean!
A few nits. Mainly, I would not use this PR to introduce hooks. Hooks are a form of meta-programming, adding more overhead. When they are consistently used, they might provide an overall benefit, but since the maps
plugin does not use them right now, and since their use here is not saving any LOC here, I wouldn't make this refactor.
x-pack/plugins/maps/public/classes/sources/es_geo_grid_source/render_as_select.tsx
Outdated
Show resolved
Hide resolved
x-pack/plugins/maps/public/classes/sources/es_geo_grid_source/render_as_select.tsx
Outdated
Show resolved
Hide resolved
x-pack/plugins/maps/public/classes/sources/es_geo_grid_source/render_as_select.tsx
Outdated
Show resolved
Hide resolved
x-pack/plugins/maps/public/classes/sources/es_geo_grid_source/render_as_select.tsx
Outdated
Show resolved
Hide resolved
x-pack/plugins/maps/public/classes/sources/es_geo_grid_source/render_as_select.tsx
Outdated
Show resolved
Hide resolved
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.
LGTM
code review, tested in chrome
Pinging @elastic/kibana-gis (Team:Geo) |
@elasticmachine merge upstream |
buildkite test this |
@elasticmachine merge upstream |
💚 Build Succeeded
Metrics [docs]Async chunks
History
To update your PR or re-run it, just comment with: |
Closes: #121897
This PR contains changes for the "Clusters and grids" feature.
Users will now be able to see all the viewing options and select it from a group of buttons, rather than from a drop-down list as before.
How it will looks like:
cc @nreese