Michael Coker

Hey @Marc19 and @warmbowski, thanks for the issue. Just wanted to check in and say I'm hoping to have a look at this tomorrow and see if there is anything...

@eliranmal of course, thanks for taking it on!!

@eliranmal my apologies for the delay! I'm still seeing it here - https://www.patternfly.org/v4/components/data-list/html#selectable-rows I'm using chrome, and the top shadow is visible on the "secondary content" item by default *...

@mmenestr my $0.02 about the default behavior - when hitting enter or clicking the search button is what updates the menu (versus the menu updating as you type/change the text...

> Another question @mcoker, should we have a clear button like we had in previous implementation? @tlabaj I think it's probably useful. I don't see any design guidelines around when...

We noticed this inconsistency while updating the custom icons in core, and how react-icons scale down the icon height when the icon is wider than it is tall. An example...

@mcarrano good question... I don't see why it would necessarily need to be tied to the notification drawer. It's basically just a plain variant button with unique background/text color. It...

I'm probably doing something wrong, but it doesn't seem to be doing anything for me. I branched from this PR branch and was able to commit without a valid semantic...

So that it will wrap with the text, it's probably best just to put it inline with the step content and apply a margin. Here I've used the `sm` margin,...

> I do believe that's happening because the timeout doesn't get triggered to start until you move focus off the tooltip, or mouseout of the tooltip. I think that's intended?...