Skip to content

fix(form): keep required asterisk from orphaning#11961

Merged
nicolethoen merged 2 commits into
patternfly:mainfrom
mcoker:issue-11960
Dec 11, 2025
Merged

fix(form): keep required asterisk from orphaning#11961
nicolethoen merged 2 commits into
patternfly:mainfrom
mcoker:issue-11960

Conversation

@mcoker
Copy link
Copy Markdown
Contributor

@mcoker mcoker commented Aug 12, 2025

fixes #11960

@mcoker mcoker requested a review from kmcfaul August 12, 2025 16:01
@patternfly-build
Copy link
Copy Markdown
Collaborator

patternfly-build commented Aug 12, 2025

@github-actions
Copy link
Copy Markdown

This PR has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@github-actions github-actions Bot added the Stale label Oct 15, 2025
@mcoker mcoker marked this pull request as ready for review November 12, 2025 18:24
@mcoker
Copy link
Copy Markdown
Contributor Author

mcoker commented Nov 12, 2025

Reviving this old PR. Just replaces the space char between the label and asterisk with a non-breaking space. This is the markup it renders. I'm not sure where the html comment comes from, but it doesn't impact anything.

Screenshot 2025-11-12 at 12 27 32 PM

Same concept as how it's done in core

Screenshot 2025-11-12 at 12 25 48 PM

Copy link
Copy Markdown
Member

@srambach srambach left a comment

Choose a reason for hiding this comment

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

Works for me (unless I put a space as the last character of the label 😈 which just don't do that!)

@github-actions github-actions Bot removed the Stale label Nov 16, 2025
@nicolethoen nicolethoen merged commit d11c186 into patternfly:main Dec 11, 2025
13 checks passed
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 - FormGroup- if using isRequired prop - the "*" can show an a separate line

5 participants