Michael Coker
Michael Coker
Yeah! I think my original goal with this was a couple of things: * Call out that chart styles work out of the box without any additional CSS imports, except...
closed https://github.com/patternfly/patternfly/issues/6390
Closing for now as the colors are already generated
@thatblindgeye would you mind re-reviewing when you get a chance? Added `aria-hidden` and `tabindex="-1"` (matches the button and other places we're disabling links/non-buttons), which I just missed when making this...
cc @andrew-ronaldson @kaylachumley just to confirm, a disabled element's status icon should retain its status color? I poked through design guidelines and didn't see anything that called this out specifically....
dupe of https://github.com/patternfly/patternfly/issues/7183
dupe of https://github.com/patternfly/patternfly/issues/7699
For bottom pagination, when going between mobile and desktop, the order of the buttons/menus/etc changes, which makes the tab order not match the order of the elements in the code...
Just want to +1 removing `` unless it's needed. Also if it's used, you'll need to pass `isInline` so that it inherits font size/color. You can see the icons here...
@srambach good catches! > @mcoker does this qualify as a breaking change? We've added pf-v6-c-button and pf-m-plain which I think would be ok? But also removed pf-v6-c-data-list__item-draggable-icon The button classes...