feat(Masthead): Updates for new masthead structure in Penta by tlabaj · Pull Request #10809

feat(Masthead): Updates for new masthead structure in Penta by tlabaj · Pull Request #10809 · patternfly/patternfly-react

Conversation

What: Closes #10788

Renamed MastheadBrand to MastheadLogo
Renamed MastheadMain to MastheadBrand
Wrap MastheadToggle and MastheadBrand in MastheadMain

Codemod issue: patternfly/pf-codemods#718

tlabaj linked an issue

Jul 26, 2024

that may be closed by this pull request

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Surge is down so I haven't run it yet but had a couple of questions. Otherwise looks good on inspection!

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

since surge is down, this is going off the file changes as best I can. can re-review if/when surge comes back to make sure I checked everything, but this probably covers the main things regardless!

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm. We may want to open a follow up in org if we don't already have one to update the structure for the docs site (the surge masthead is the old structure still).