-
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
[Dashboard] Compressed Styles for Controls #190636
Conversation
Pinging @elastic/kibana-presentation (Team:Presentation) |
src/plugins/controls/public/react_controls/control_group/components/control_group_editor.tsx
Outdated
Show resolved
Hide resolved
src/plugins/controls/public/control_group/component/control_frame_component.tsx
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.
...s/dashboard/public/dashboard_app/top_nav/controls_toolbar_button/controls_toolbar_button.tsx
Outdated
Show resolved
Hide resolved
src/plugins/controls/public/range_slider/components/range_slider_editor_options.tsx
Outdated
Show resolved
Hide resolved
Main now uses react embeddable control group and we are activity working on removing the legacy control group. Changes to legacy control group should be discarded as they are not needed. |
src/plugins/controls/public/control_group/component/control_frame_component.tsx
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.
Tested locally. Design changes LGTM, thanks!
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.
Changes LGTM 👍 Tested locally + code review.
💚 Build Succeeded
Metrics [docs]Async chunks
Page load bundle
History
To update your PR or re-run it, just comment with: cc @rshen91 |
…ols on Alerts Page. (#192094) ## Summary As stated in #192092 , the labels of filter controls were mis-aligned. This was due to some changes in EUI were conflicting with our custom styling. This PR removes the custom styling. ### Before ![grafik](https://github.com/user-attachments/assets/f167f909-6fff-442e-9ed8-a9781bb52bd1) ### Issues with Error state It looks like recent PR #190636 on 03-Sept made the controls compact but the error state was left out. This PR fixes that. I would request @elastic/kibana-presentation team to review it once. <img width="1920" alt="grafik" src="https://github.com/user-attachments/assets/4025c79f-2602-4221-a136-436b6759096f"> ### After Below video shows fixes in success, loading and error states. @elastic/response-ops team, Could you please check in other areas as well where these filters are used. Thanks. https://github.com/user-attachments/assets/7e0f6035-171c-4a4b-99c6-116ad577a019
…ols on Alerts Page. (elastic#192094) ## Summary As stated in elastic#192092 , the labels of filter controls were mis-aligned. This was due to some changes in EUI were conflicting with our custom styling. This PR removes the custom styling. ### Before ![grafik](https://github.com/user-attachments/assets/f167f909-6fff-442e-9ed8-a9781bb52bd1) ### Issues with Error state It looks like recent PR elastic#190636 on 03-Sept made the controls compact but the error state was left out. This PR fixes that. I would request @elastic/kibana-presentation team to review it once. <img width="1920" alt="grafik" src="https://github.com/user-attachments/assets/4025c79f-2602-4221-a136-436b6759096f"> ### After Below video shows fixes in success, loading and error states. @elastic/response-ops team, Could you please check in other areas as well where these filters are used. Thanks. https://github.com/user-attachments/assets/7e0f6035-171c-4a4b-99c6-116ad577a019
Summary
Closes #190355
This builds off @ryankeairns work in #189839 to have the compressed styles for controls
Applied to new controls in example apps:
Applied to existing dashboard controls: