Michael Coker
Michael Coker
@evwilkin nice, thank you! Looks like we can transfer this to core 👍
> or PageSection with type breadcrumb, do we still intend for a PageSection component to have the breadcrumb class applied? @thatblindgeye yep - as far as I can tell those...
Closing for now, can re-open issues as needed if bugs/tasks come up
@GregJohnStewart thanks for the issue 👍 I'm in favor of making it easier to get the styles/assets from core in case you're not using node/npm. WDYT @nicolethoen @dlabaj? Do you...
Closing for the time being, we're past the point of making breaking changes and have made a bunch of improvements since this issue was created. AFAIK there is nothing in...
Menu toggle supports success, warning, and danger
@mcarrano one issue is that currently the background changes if you hover over the dead space, which is an accessibility concern. The background should only change if you're hovered/focused on...
@blfetche you got it. Ideally on a single-line row/node (no wrapping), there is no space above/below the toggle on the left and the actions on the right. And per @mmenestr's...
@mcarrano works for me from a technical standpoint. Not sure if we want to get anyone else's opinion on the a11y since the background change on hover is usually tied...
Can revisit if it's reported as an issue.