-
Notifications
You must be signed in to change notification settings - Fork 376
fix(CodeEditor): Set default height #11014
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
Conversation
|
Preview: https://patternfly-react-pr-11014.surge.sh A11y report: https://patternfly-react-pr-11014-a11y.surge.sh |
daec7e7 to
4b7c712
Compare
cd0543f to
7dfb53c
Compare
824a30b to
2f06af0
Compare
The docs state that the default height is 100%. However, the examples just break if you don't pass in a height. This should help.
2f06af0 to
ae62ef8
Compare
kmcfaul
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🎉
|
This juSt needs core review @mcoker |
mcoker
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
|
Your changes have been released in:
Thanks for your contribution! 🎉 |
The docs state that the default height for CodeEditor is 100%. However, the examples just break if you don't pass in a height. I got it to work with these changes.
We just need to upgrade Core once a release comes out that contains these classes.
What: Closes #11013
Additional issues:
Related to patternfly/patternfly#7087