Michael Coker comments

Results 156 comments of


                                            Michael Coker

fix(CodeEditor): use codeEditorControls and clean up overall

> Given that the 'Copied' text is on its own timer, and it does not get reset when you do things like move your mouse away from the trigger or...

feat(Icon): add Icon component

@tlabaj > Is there a reason the spinner and the icon have to render at the same time Link in the core example? is it to maintain space when the...

feat(OverflowMenu): allow breakpoint on container width

@thatblindgeye just thinking out loud a bit: I wonder if it might make sense to support 2 things to basically manage the behavior. 1. A _thing_ to watch the size...

Consider using panel for menu dialog boxes and for placing content above/below the menu

Removing from breaking change milestone - we decided to make this a menu next component post v5 release.

Wizard - support for step status

@mcarrano we can always add them later if we don't have use cases for other statuses.

Button order for the Wizard is counter intuitive/not following established convention

cc @mceledonia @mcarrano

Button order for the Wizard is counter intuitive/not following established convention

closed via https://github.com/patternfly/patternfly/pull/5148

Bug - TreeView - Can not put LabelGroup in expandable TreeViewItem due to nested button elements

cc @mcarrano @mmenestr wdyt?

fix(Slider): reorder slider thumb and slider input elements

@thatblindgeye and I discussed and I think a consideration for this should be that the floating input is currently more like a popover/tooltip attached to the thumb. I can see...

Investigate using :is()

Hey @garrett! Yeah, we started using `:where()` with dark theme since it seemed like the best way to handle the overrides without introducing specificity issues, and it was added to...