-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Discover] Fix theme for Alerts popover #145390
[Discover] Fix theme for Alerts popover #145390
Conversation
Pinging @elastic/kibana-data-discovery (Team:DataDiscovery) |
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.
src/plugins/discover/public/application/main/components/top_nav/open_alerts_popover.tsx
Outdated
Show resolved
Hide resolved
…v/open_alerts_popover.tsx Co-authored-by: Matthias Wilhelm <ankertal@gmail.com>
@elasticmachine merge upstream |
💚 Build Succeeded
Metrics [docs]Async chunks
Unknown metric groupsESLint disabled in files
ESLint disabled line counts
Total ESLint disabled count
History
To update your PR or re-run it, just comment with: cc @jughosta |
Closes elastic#143070 ## Summary This PR adds theme provider to the Alerts popover. Dark theme: <img width="534" alt="Screenshot 2022-11-16 at 16 37 49" src="https://user-images.githubusercontent.com/1415710/202226249-a46f6608-4f35-40ce-bfbf-847e9a7ba81a.png"> Light theme: <img width="531" alt="Screenshot 2022-11-16 at 16 38 17" src="https://user-images.githubusercontent.com/1415710/202226260-9e67a8dd-f6a4-40ff-ae28-2691b9a082bf.png"> Co-authored-by: Matthias Wilhelm <ankertal@gmail.com> (cherry picked from commit e33392b)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
# Backport This will backport the following commits from `main` to `8.6`: - [[Discover] Fix theme for Alerts popover (#145390)](#145390) <!--- Backport version: 8.9.7 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Julia Rechkunova","email":"julia.rechkunova@elastic.co"},"sourceCommit":{"committedDate":"2022-11-17T21:32:39Z","message":"[Discover] Fix theme for Alerts popover (#145390)\n\nCloses https://github.com/elastic/kibana/issues/143070\r\n\r\n## Summary\r\n\r\nThis PR adds theme provider to the Alerts popover.\r\n\r\nDark theme:\r\n<img width=\"534\" alt=\"Screenshot 2022-11-16 at 16 37 49\"\r\nsrc=\"https://user-images.githubusercontent.com/1415710/202226249-a46f6608-4f35-40ce-bfbf-847e9a7ba81a.png\">\r\n\r\nLight theme:\r\n<img width=\"531\" alt=\"Screenshot 2022-11-16 at 16 38 17\"\r\nsrc=\"https://user-images.githubusercontent.com/1415710/202226260-9e67a8dd-f6a4-40ff-ae28-2691b9a082bf.png\">\r\n\r\nCo-authored-by: Matthias Wilhelm <ankertal@gmail.com>","sha":"e33392bff0a022bb246f8d560a336ebdac0555f9","branchLabelMapping":{"^v8.7.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix","Team:DataDiscovery","backport:prev-minor","v8.7.0"],"number":145390,"url":"https://github.com/elastic/kibana/pull/145390","mergeCommit":{"message":"[Discover] Fix theme for Alerts popover (#145390)\n\nCloses https://github.com/elastic/kibana/issues/143070\r\n\r\n## Summary\r\n\r\nThis PR adds theme provider to the Alerts popover.\r\n\r\nDark theme:\r\n<img width=\"534\" alt=\"Screenshot 2022-11-16 at 16 37 49\"\r\nsrc=\"https://user-images.githubusercontent.com/1415710/202226249-a46f6608-4f35-40ce-bfbf-847e9a7ba81a.png\">\r\n\r\nLight theme:\r\n<img width=\"531\" alt=\"Screenshot 2022-11-16 at 16 38 17\"\r\nsrc=\"https://user-images.githubusercontent.com/1415710/202226260-9e67a8dd-f6a4-40ff-ae28-2691b9a082bf.png\">\r\n\r\nCo-authored-by: Matthias Wilhelm <ankertal@gmail.com>","sha":"e33392bff0a022bb246f8d560a336ebdac0555f9"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v8.7.0","labelRegex":"^v8.7.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/145390","number":145390,"mergeCommit":{"message":"[Discover] Fix theme for Alerts popover (#145390)\n\nCloses https://github.com/elastic/kibana/issues/143070\r\n\r\n## Summary\r\n\r\nThis PR adds theme provider to the Alerts popover.\r\n\r\nDark theme:\r\n<img width=\"534\" alt=\"Screenshot 2022-11-16 at 16 37 49\"\r\nsrc=\"https://user-images.githubusercontent.com/1415710/202226249-a46f6608-4f35-40ce-bfbf-847e9a7ba81a.png\">\r\n\r\nLight theme:\r\n<img width=\"531\" alt=\"Screenshot 2022-11-16 at 16 38 17\"\r\nsrc=\"https://user-images.githubusercontent.com/1415710/202226260-9e67a8dd-f6a4-40ff-ae28-2691b9a082bf.png\">\r\n\r\nCo-authored-by: Matthias Wilhelm <ankertal@gmail.com>","sha":"e33392bff0a022bb246f8d560a336ebdac0555f9"}}]}] BACKPORT--> Co-authored-by: Julia Rechkunova <julia.rechkunova@elastic.co>
Closes #143070
Summary
This PR adds theme provider to the Alerts popover.
Dark theme:
Light theme: