Patternfly-react: Latest PF4 packages: tab content

Created on 16 Apr 2019  路  5Comments  路  Source: patternfly/patternfly-react

Installed the latest Patternfly and patternfly-react packages and the Cost Management UI is looking odd.

In this case, there is a problem with the tab content.

When the page is loaded, progress bars do not expand the entire width of the card body as expected. As a result, our Insights skeletons and PF progress bars no longer stretch the entire width.

It appears there is an issue with the select element output by the tabs component? I can override that to 100% and the progress bars fill the card.

Note that this also affects our use of Insights skeleton components. The skeletons look fine in other areas -- seems to be just an issue with the tabs?

Progress bars before:
Screen Shot 2019-04-16 at 11 42 42 AM

Progress bars after:
SAVE

Skeletons before:
Screen Shot 2019-04-16 at 11 46 42 AM

Skeletons after:
Screen Shot 2019-04-15 at 5 28 40 PM (2)

P0 PF4 rc.1

Most helpful comment

@tlabaj looks like this is the same issue as https://github.com/patternfly/patternfly-react/issues/1775

All 5 comments

@dlabrecq are you using the TabContent Outside of the tabs here?

@tlabaj looks like this is the same issue as https://github.com/patternfly/patternfly-react/issues/1775

I'm not using TabContent here, just Tabs and Tab.

@tlabaj looking at the old implementation of tabs, the tab content sections are outside of the tabs component parent. With Dan's latest updates, the tab content sections are now inside of the tabs. Do you know when and why that changed? I'm curious if I need to provide a CSS fix or if this is something we can sort out in the react component.

Previous

Screen Shot 2019-04-16 at 4 38 38 PM

Current

Screen Shot 2019-04-16 at 4 45 15 PM

@mcoker It probably changed with the refactoring for the TabContent outside of Tabs. . I can make that change so it is outside of the tabs again.

Was this page helpful?
0 / 5 - 0 ratings