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(form): dont add whitespace on empty required field labels #3060

Merged
merged 4 commits into from
Jul 6, 2024

Conversation

lubber-de
Copy link
Member

@lubber-de lubber-de commented Jun 19, 2024

Description

Followup of #2711

Whenever an empty label was used for a required single field or non-grouped, non-inline fields group, the label was still receiving an extra space to create a default height and did not display the asterisk. The intention of the space character was for empty non required fields to match same height when used inside a fields row.

If one does not want a label at all and does not want to make this field required, you would not render the label at all and wont get additional space then

Closes

#2711 (comment)

@lubber-de lubber-de added type/bug Any issue which is a bug or PR which fixes a bug lang/css Anything involving CSS state/awaiting-reviews Pull requests which are waiting for reviews labels Jun 19, 2024
@lubber-de lubber-de added this to the 2.9.4 milestone Jun 19, 2024
@mvorisek
Copy link
Contributor

In my original https://jsfiddle.net/h9ta2wdj/ the space is added for both required/non-required fields with empty label, can you edit the CSS in my repro to show what this change would do? I think the change is not fully enough.

@lubber-de
Copy link
Member Author

@mvorisek Thanks for the feedback
I think, i fixed it now
See your adjusted fsfiddle here https://jsfiddle.net/lubber/owayxz3k/22/

@lubber-de lubber-de merged commit 3db5328 into fomantic:develop Jul 6, 2024
10 checks passed
@lubber-de lubber-de deleted the emptyRequiredLabelAsterisk branch July 6, 2024 18:54
@lubber-de lubber-de removed the state/awaiting-reviews Pull requests which are waiting for reviews label Jul 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lang/css Anything involving CSS type/bug Any issue which is a bug or PR which fixes a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants