Skip to content

fix(CodeEditor): do not include extra div if headerMainContent is unset#11534

Merged
thatblindgeye merged 1 commit intopatternfly:mainfrom
logonoff:codeeditor2
Feb 17, 2025
Merged

fix(CodeEditor): do not include extra div if headerMainContent is unset#11534
thatblindgeye merged 1 commit intopatternfly:mainfrom
logonoff:codeeditor2

Conversation

@logonoff
Copy link
Copy Markdown
Member

What: Closes #11533

@patternfly-build
Copy link
Copy Markdown
Collaborator

patternfly-build commented Feb 13, 2025

@kmcfaul
Copy link
Copy Markdown
Contributor

kmcfaul commented Feb 14, 2025

Looks like this just needs a snapshot update for the failing test (since the extra div is not included anymore). You should be able to update it with yarn run jest CodeEditor -u (or just yarn test --u but that runs the whole suite).

@thatblindgeye thatblindgeye merged commit 86ff923 into patternfly:main Feb 17, 2025
13 checks passed
@logonoff logonoff deleted the codeeditor2 branch February 17, 2025 14:26
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] - pf-v6-c-code-editor__header-main exists even when headerMainContent is an empty string

5 participants