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

[PLAY-1703 ] Form Group Error State Alignment #4004

Merged
merged 6 commits into from
Dec 26, 2024

Conversation

Tomm1128
Copy link
Contributor

@Tomm1128 Tomm1128 commented Dec 11, 2024

What does this PR do? A clear and concise description with your runway ticket url.
Fixes the alignments of the form group items when there are validation errors being displayed on the kit and when a label is used in some of the form group kits.

PLAY-1703

Screenshots: Screenshots to visualize your addition/change

Screenshot 2024-12-11 at 10 19 36 AM Screenshot 2024-12-11 at 10 19 54 AM Screenshot 2024-12-11 at 10 20 52 AM

How to test? Steps to confirm the desired behavior:

  1. Go to 'kits/form_group
  2. Scroll down to the phone number kit
  3. Click into the kit and then click out, should see validation error message in red around the kit
  4. The form group should still be aligned with and without the error.
  5. Should work for both React and Rails

Checklist:

  • LABELS Add a label: enhancement, bug, improvement, new kit, deprecated, or breaking. See Changelog & Labels for details.
  • DEPLOY I have added the milano label to show I'm ready for a review.
  • TESTS I have added test coverage to my code.

@Tomm1128 Tomm1128 added bug Fixes to issues discovered in Playbook (USED IN CHANGELOG) minor Semver Target labels Dec 11, 2024
@Tomm1128 Tomm1128 self-assigned this Dec 11, 2024
@Tomm1128 Tomm1128 requested a review from a team as a code owner December 11, 2024 15:23
@Tomm1128 Tomm1128 added milano 20 MAX - Deploy this PR to a review environment via Milano Needs Review labels Dec 11, 2024
jasperfurniss
jasperfurniss previously approved these changes Dec 11, 2024
Copy link

🎉 Congratulations on creating an Alpha Version!

Your Alpha for Ruby Gems is 14.9.0.pre.alpha.play1703errorstatealignment4889

Your Alpha for NPM is 14.9.0-alpha.play1703errorstatealignment4889

@nidaqg nidaqg added Code Approved Approved by a Playbook Admin and removed Needs Review labels Dec 12, 2024
@Tomm1128 Tomm1128 added Needs Review and removed Code Approved Approved by a Playbook Admin labels Dec 18, 2024
@Tomm1128 Tomm1128 added alpha and removed alpha labels Dec 18, 2024
Copy link

🎉 Congratulations on creating an Alpha Version!

Your Alpha for Ruby Gems is 14.9.0.pre.alpha.play1703errorstatealignment4991

Your Alpha for NPM is 14.9.0-alpha.play1703errorstatealignment4991

jasperfurniss
jasperfurniss previously approved these changes Dec 19, 2024
@jasperfurniss jasperfurniss added Code Approved Approved by a Playbook Admin and removed Needs Review labels Dec 19, 2024
Copy link

🎉 Congratulations on creating an Alpha Version!

Your Alpha for Ruby Gems is 14.9.0.pre.alpha.play1703errorstatealignment5060

Your Alpha for NPM is 14.9.0-alpha.play1703errorstatealignment5060

@jasperfurniss jasperfurniss added Code Approved Approved by a Playbook Admin and removed Needs Review labels Dec 20, 2024
@nidaqg nidaqg added the Product Approved pending technical review, OK to merge to master label Dec 26, 2024
@nidaqg nidaqg merged commit 4c3f9b6 into master Dec 26, 2024
16 checks passed
@nidaqg nidaqg deleted the play-1703-error-state-alignment branch December 26, 2024 13:49
Copy link

You merged this pr to master branch:
- Ruby Gem: 14.10.0.pre.rc.24
- NPM: 14.10.0-rc.24

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
alpha bug Fixes to issues discovered in Playbook (USED IN CHANGELOG) Code Approved Approved by a Playbook Admin milano 20 MAX - Deploy this PR to a review environment via Milano minor Semver Target Product Approved pending technical review, OK to merge to master
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants