Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(page): default page to full viewport height #5517

Merged
merged 1 commit into from May 2, 2023

Conversation

srambach
Copy link
Member

Fixes #5448
Removes .pf-m-full-height for a page in lieu of defaulting the page to be 100vh/dvh with a max-height so it doesn't overrun the space it's allocated if it's within another layout.

This will need a follow-on issue on .org to limit the width of all the page examples (this is currently done just for a couple of the examples).

@patternfly-build
Copy link

patternfly-build commented Apr 27, 2023

Copy link
Contributor

@mcoker mcoker left a comment

Choose a reason for hiding this comment

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

👍

@mcoker mcoker merged commit abcd907 into patternfly:v5 May 2, 2023
1 check passed
@mcoker mcoker changed the title chore(page): default page to full viewport height fix(page): default page to full viewport height May 2, 2023
@patternfly-build
Copy link

🎉 This PR is included in version 5.0.0-alpha.44 🎉

The release is available on:

Your semantic-release bot 📦🚀

mattnolting pushed a commit to mattnolting/patternfly that referenced this pull request May 18, 2023
mattnolting pushed a commit to mattnolting/patternfly that referenced this pull request Dec 12, 2023
@srambach srambach deleted the 5448-default-full-screen branch April 6, 2024 02:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Page - make pf-m-full-height the default
3 participants