Skip to content

Conversation

@rebeccaalpert
Copy link
Member

@rebeccaalpert rebeccaalpert commented Sep 17, 2024

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

@patternfly-build
Copy link
Collaborator

patternfly-build commented Sep 17, 2024

@rebeccaalpert rebeccaalpert changed the base branch from main to v6 September 17, 2024 18:29
@rebeccaalpert rebeccaalpert changed the base branch from v6 to main September 17, 2024 18:29
@rebeccaalpert rebeccaalpert force-pushed the code-editor-height branch 3 times, most recently from daec7e7 to 4b7c712 Compare September 18, 2024 18:14
@rebeccaalpert rebeccaalpert force-pushed the code-editor-height branch 2 times, most recently from 824a30b to 2f06af0 Compare November 8, 2024 18:18
@rebeccaalpert rebeccaalpert changed the title Draft: fix(CodeEditor): Set default height fix(CodeEditor): Set default height Nov 8, 2024
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.
Copy link
Contributor

@kmcfaul kmcfaul left a comment

Choose a reason for hiding this comment

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

LGTM 🎉

@tlabaj tlabaj requested a review from mcoker November 13, 2024 14:37
@tlabaj
Copy link
Contributor

tlabaj commented Nov 13, 2024

This juSt needs core review @mcoker

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.

🚀

@tlabaj tlabaj merged commit 6ddfc93 into patternfly:main Nov 13, 2024
10 checks passed
@patternfly-build
Copy link
Collaborator

Your changes have been released in:

  • @patternfly/react-charts@8.1.0-prerelease.3
  • @patternfly/react-code-editor@6.1.0-prerelease.2
  • @patternfly/react-core@6.1.0-prerelease.2
  • @patternfly/react-docs@7.1.0-prerelease.3
  • @patternfly/react-drag-drop@6.1.0-prerelease.2
  • @patternfly/react-icons@6.1.0-prerelease.1
  • demo-app-ts@6.0.0-prerelease.30
  • @patternfly/react-styles@6.1.0-prerelease.1
  • @patternfly/react-table@6.1.0-prerelease.2
  • @patternfly/react-templates@6.1.0-prerelease.2
  • @patternfly/react-tokens@6.1.0-prerelease.1

Thanks for your contribution! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug - CodeEditor - Default height is not 100%

5 participants