elastic
Add toolbar button styles
Perhaps this could instead be a discussion topic, but Lens and the presentation toolbar (Dashboard, Canvas) are using the same toolbar button styles which smells like a possible EUI addition (e.g. new button group variation?).
They've been operating with separate sets of styles and, as of this writing, are having to make simultaneous adjustments due to the Amsterdam switch in order to gain more contrast with the underlying background color.
The latest design proposal is here (adding a border):
Thanks @ryankeairns,
For sharing this. The Maps app also has toolbar buttons.
I think in EUI we could definitely create a toolbar button and toolbar button groups to create some consistency across all apps.
👋 Hey there. This issue hasn't had any activity for 180 days. We'll automatically close it if that trend continues for another week. If you feel this issue is still valid and needs attention please let us know with a comment.
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
This issue is still valid, but it would be helpful if the EUI team had a more thorough design spec to go off of. @ryankeairns Is there someone on your team who is familiar with all the states/configurations that can build this out in Figma?
@cchaos yep, @MichaelMarcialis can assist with that although it won't be real soon. Let's consider that a next step and he'll get to it as his workload allows.
@MichaelMarcialis and @ryankeairns - do y'all still wanna get this implemented in EUI?
From my perspective, I believe it still makes sense to support this at the EUI level. The current landscape is a bit all over the place:
- Lens uses the
ToolbarButtonshared component in Kibana. - Dashboard and Canvas use a mix of a deprecated
QuickButtonGroupcomponent alongsideEuiButtoncomponents with style overrides. - Maps also uses its own set of toolbar button markup and styles.
In any case, either a new component or enhancement to the existing button components at the EUI level would likely help provide better consistency across the apps and help us retire some of these redundant, bespoke solutions.
👋 Hi there - this issue hasn't had any activity in 6 months. If the EUI team has not explicitly expressed that this is something on our roadmap, it's unlikely that we'll pick this issue up. We would sincerely appreciate a PR/community contribution if this is something that matters to you! If not, and there is no further activity on this issue for another 6 months (i.e. it's stale for over a year), the issue will be auto-closed.
I believe this is still a relevant issue, and I'm seeing that I probably owe ya'll some additional spec details. I'll put it on my to-do list to write something up as soon as I'm able.
👋 Hi there - this issue hasn't had any activity in 6 months. If the EUI team has not explicitly expressed that this is something on our roadmap, it's unlikely that we'll pick this issue up. We would sincerely appreciate a PR/community contribution if this is something that matters to you! If not, and there is no further activity on this issue for another 6 months (i.e. it's stale for over a year), the issue will be auto-closed.