-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
Make "experimental" banner collapsable/hidable. #15713
Comments
@snide @cjcenizal @cchaos I'd like your feedback here. We'd like to reduce the amount of space for the warning we present for lab visualizations. Not only that, but is there a better way we can style it? I know the ask here is to make the banner collapsable, but I'm wondering if we reduce the amount of space being used if it's still necessary. cc: @thomasneirynck |
I agree just making it smaller would be enough imho |
++ maybe we can sneak this into 6.2? |
Can this be surfaced using |
I would just suggest switching to use the I'd also suggest adding the experimental icon |
I'm just seeing @cjcenizal's comment. That could work as well, but if you just want a quick component swap, you could go with the It's also worth considering if this is something that should be dismiss-able at all, or at least user-dismissed vs auto-dismissed. |
I like the I would vote against the info notification, since I think they are (the way they currently behave) a huge disturbance for the user and offer bad usability (they can stay when changing the screen, but doesn't need to, sometimes I see a banner up, and am not sure if it just appeared again, or is the one from earlier, etc.). So I would rather go with |
We can do that. I'll update the component to add a |
There is now a small version of the EuiCallOut available. |
The experimental banner takes up considerable screen real estate, without being able to collapse/hide it.
The text was updated successfully, but these errors were encountered: