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): preserve intrinsic block size of field labels #5639

Merged
merged 3 commits into from
Feb 5, 2024

Conversation

juice49
Copy link
Contributor

@juice49 juice49 commented Feb 1, 2024

Description

This branch improves vertical alignment in boolean fields. It uses CSS size containment to prevent the block size of elements appended to field labels (such as the deprecated field badge) affecting the intrinsic block size of the label.

Closes #5579.

Before (non-deprecated booleans)

before

After (non-deprecated booleans)

after

Before (deprecated booleans)

before-deprecated

After (deprecated booleans)

after-deprecated

What to review

  • Does the vertical alignment in boolean fields look better?
  • Does general field rendering still look correct?

Testing

  • Take a look at the Test Studio. In particular, booleans (/test/structure/input-standard;booleansTest).

Notes for release

Improve vertical alignment of boolean fields.

Copy link

vercel bot commented Feb 1, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
performance-studio ✅ Ready (Inspect) Visit Preview Feb 1, 2024 11:46am
test-studio ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 1, 2024 11:46am
1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
studio-workshop ⬜️ Ignored (Inspect) Visit Preview Feb 1, 2024 11:46am

Copy link
Contributor

github-actions bot commented Feb 1, 2024

No changes to documentation

Copy link
Contributor

github-actions bot commented Feb 1, 2024

Component Testing Report Updated Feb 1, 2024 11:50 AM (UTC)

File Status Duration Passed Skipped Failed
comments/CommentInput.spec.tsx ✅ Passed (Inspect) 33s 15 0 0
formBuilder/ArrayInput.spec.tsx ✅ Passed (Inspect) 6s 3 0 0
formBuilder/inputs/PortableText/Annotations.spec.tsx ✅ Passed (Inspect) 13s 3 0 0
formBuilder/inputs/PortableText/copyPaste/CopyPaste.spec.tsx ✅ Passed (Inspect) 13s 4 2 0
formBuilder/inputs/PortableText/Decorators.spec.tsx ✅ Passed (Inspect) 13s 6 0 0
formBuilder/inputs/PortableText/FocusTracking.spec.tsx ✅ Passed (Inspect) 33s 15 0 0
formBuilder/inputs/PortableText/Input.spec.tsx ✅ Passed (Inspect) 18s 9 0 0
formBuilder/inputs/PortableText/ObjectBlock.spec.tsx ✅ Passed (Inspect) 1m 3s 18 0 0
formBuilder/inputs/PortableText/Styles.spec.tsx ✅ Passed (Inspect) 14s 6 0 0
formBuilder/inputs/PortableText/Toolbar.spec.tsx ✅ Passed (Inspect) 20s 9 0 0

@juice49 juice49 force-pushed the fix/sdx-982-boolean-field-alignment branch from e866a90 to 9ce1eeb Compare February 1, 2024 11:28
@juice49 juice49 marked this pull request as ready for review February 1, 2024 11:28
@juice49 juice49 requested a review from a team as a code owner February 1, 2024 11:28
@juice49 juice49 requested review from bjoerge, a team and jtpetty and removed request for a team February 1, 2024 11:28
@jtpetty
Copy link
Contributor

jtpetty commented Feb 1, 2024

@juice49 - any idea why the failed check?

@juice49
Copy link
Contributor Author

juice49 commented Feb 1, 2024

@jtpetty Which failure? I believe they're all passing for the latest commit, but I might be missing something!

Update: Oh! Are you referring to this one?

[chromium] › desk/deprecatedFields.spec.ts:26:7 › slug type shows deprecated message

@juice49
Copy link
Contributor Author

juice49 commented Feb 1, 2024

@jtpetty This appears to be an issue with flaky tests. All passing on re-run 👍.

Copy link
Contributor

@binoy14 binoy14 left a comment

Choose a reason for hiding this comment

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

thank you! LGTM

@juice49 juice49 removed the request for review from bjoerge February 2, 2024 09:15
Copy link
Contributor

@jtpetty jtpetty left a comment

Choose a reason for hiding this comment

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

👍

@juice49 juice49 added this pull request to the merge queue Feb 5, 2024
Merged via the queue into next with commit 08f6694 Feb 5, 2024
41 checks passed
@juice49 juice49 deleted the fix/sdx-982-boolean-field-alignment branch February 5, 2024 09:09
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.

Studio UI: Vertical text alignment issue in checkbox-type boolean fields
3 participants