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] fix toolbar action button not filled when selected #158284
Conversation
Pinging @elastic/kibana-presentation (Team:Presentation) |
@elasticmachine merge upstream |
x-pack/plugins/maps/public/connected_components/toolbar_overlay/_toolbar_overlay.scss
Show resolved
Hide resolved
…tor/tooltip_selector.tsx
…-ref HEAD~1..HEAD --fix'
💚 Build Succeeded
Metrics [docs]Async chunks
Unknown metric groupsESLint disabled line counts
Total ESLint disabled count
History
To update your PR or re-run it, just comment with: |
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, code changes look good to me.
…8284) Fixes elastic#158221 Updated CSS selector to look for any class containing "euiButtonIcon-fill" instead of old "euiButtonIcon--fill". Had to use wild card select because class name as dynamic id in it, for example "css-1q7ycil-euiButtonIcon-empty-primary-hoverStyles" Now, button is filled and icon is white when selected. <img width="400" alt="Screen Shot 2023-05-23 at 10 34 42 AM" src="https://github.com/elastic/kibana/assets/373691/821d95da-df7a-4779-a7fa-2ce2aa61fa7d"> --------- Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com> (cherry picked from commit 33f5bb6)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
) (#158556) # Backport This will backport the following commits from `main` to `8.8`: - [[maps] fix toolbar action button not filled when selected (#158284)](#158284) <!--- Backport version: 8.9.7 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Nathan Reese","email":"reese.nathan@elastic.co"},"sourceCommit":{"committedDate":"2023-05-26T12:20:48Z","message":"[maps] fix toolbar action button not filled when selected (#158284)\n\nFixes #158221 CSS selector to look for any class containing\r\n\"euiButtonIcon-fill\" instead of old \"euiButtonIcon--fill\". Had to use\r\nwild card select because class name as dynamic id in it, for example\r\n\"css-1q7ycil-euiButtonIcon-empty-primary-hoverStyles\"\r\n\r\nNow, button is filled and icon is white when selected.\r\n<img width=\"400\" alt=\"Screen Shot 2023-05-23 at 10 34 42 AM\"\r\nsrc=\"https://github.com/elastic/kibana/assets/373691/821d95da-df7a-4779-a7fa-2ce2aa61fa7d\">\r\n\r\n---------\r\n\r\nCo-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>","sha":"33f5bb6ba57f533b9ac894f4f6020d6c855ab1ad","branchLabelMapping":{"^v8.9.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix","Team:Presentation","Feature:Maps","v8.9.0","v8.8.1"],"number":158284,"url":"#158284 fix toolbar action button not filled when selected (#158284)\n\nFixes #158221 CSS selector to look for any class containing\r\n\"euiButtonIcon-fill\" instead of old \"euiButtonIcon--fill\". Had to use\r\nwild card select because class name as dynamic id in it, for example\r\n\"css-1q7ycil-euiButtonIcon-empty-primary-hoverStyles\"\r\n\r\nNow, button is filled and icon is white when selected.\r\n<img width=\"400\" alt=\"Screen Shot 2023-05-23 at 10 34 42 AM\"\r\nsrc=\"https://github.com/elastic/kibana/assets/373691/821d95da-df7a-4779-a7fa-2ce2aa61fa7d\">\r\n\r\n---------\r\n\r\nCo-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>","sha":"33f5bb6ba57f533b9ac894f4f6020d6c855ab1ad"}},"sourceBranch":"main","suggestedTargetBranches":["8.8"],"targetPullRequestStates":[{"branch":"main","label":"v8.9.0","labelRegex":"^v8.9.0$","isSourceBranch":true,"state":"MERGED","url":"#158284 fix toolbar action button not filled when selected (#158284)\n\nFixes #158221 CSS selector to look for any class containing\r\n\"euiButtonIcon-fill\" instead of old \"euiButtonIcon--fill\". Had to use\r\nwild card select because class name as dynamic id in it, for example\r\n\"css-1q7ycil-euiButtonIcon-empty-primary-hoverStyles\"\r\n\r\nNow, button is filled and icon is white when selected.\r\n<img width=\"400\" alt=\"Screen Shot 2023-05-23 at 10 34 42 AM\"\r\nsrc=\"https://github.com/elastic/kibana/assets/373691/821d95da-df7a-4779-a7fa-2ce2aa61fa7d\">\r\n\r\n---------\r\n\r\nCo-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>","sha":"33f5bb6ba57f533b9ac894f4f6020d6c855ab1ad"}},{"branch":"8.8","label":"v8.8.1","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}] BACKPORT--> Co-authored-by: Nathan Reese <reese.nathan@elastic.co>
Fixes #158221
Updated CSS selector to look for any class containing "euiButtonIcon-fill" instead of old "euiButtonIcon--fill". Had to use wild card select because class name as dynamic id in it, for example "css-1q7ycil-euiButtonIcon-empty-primary-hoverStyles"
Now, button is filled and icon is white when selected.