Michael Coker
Michael Coker
The question came up today about whether we could update the about modal to use the modal component (it uses its own version of a modal currently), and we discussed...
re: https://github.com/patternfly/patternfly/pull/4778/files#r848962284 Before removing `min-height: 0`, let's test and make sure this won't inadvertently cause a visual breaking change if we remove it, or see if there is possibly another...
Couple of follow ups to the demo https://github.com/patternfly/patternfly/pull/4896#issuecomment-1162389578 in https://patternfly-pr-4896.surge.sh/components/wizard/html-demos/with-drawer-in-page/, the "open drawer" link should only be present when the drawer is closed. I also noticed we limit the page...
Typically we use sass to group selectors like: ``` .pf-c-content { p, dl, ol, ul, blockquote, small, pre, table, hr { &:not(:last-child) { // This variable name doesn't reflect the...
We have this support in the current page header that allows a button in the header to have a "selected state". We should add support for that in the masthead,...
Replacing an icon with a spinner can cause layout shifts since icons are often different sizes. Let's create a way to support showing an icon and replacing it with a...
re: https://github.com/patternfly/patternfly-react/pull/7089#issuecomment-1075428624 Looks to be a gotcha with flex layouts and something with images/icons. I put together a demo of alignment issues between a toggle with an icon, image, and...
* There should be no shadow between adjacent selected rows. * There is currently a thin grey border showing between adjacent selected rows when you hover over one. * The...
From a request on slack: > Hello! I have an app following the typical Patternfly layout, with a PageSidebar theme="dark" nav={Navigation} where Navigation contains the usual Nav and NavList. What...
From https://www.patternfly.org/v4/components/drawer/html#usage-0 > Modifies the drawer panel to expand from the bottom. Note: percentage based panel sizes require the drawer component's parent element have an implicit or explicit height. This...