Skip to content

feat: Remove horizontal value on FormField and FormFieldGroup orientation prop #3254

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

Merged

Conversation

mannycarrera4
Copy link
Contributor

@mannycarrera4 mannycarrera4 commented Apr 14, 2025

Summary

Fixes: #2915

Release Category

Components

BREAKING CHANGES

horizontal is no longer a supported value to the orientation prop on FormField and FormFieldGroup. Please use horizontalStart instead. This value was deprecated in v12 and a codemod was added to rename the value.


Checklist

For the Reviewer

  • PR title is short and descriptive
  • PR summary describes the change (Fixes/Resolves linked correctly)
  • PR Release Notes describes additional information useful to call out in a release message or removed if not applicable
  • Breaking Changes provides useful information to upgrade to this code or removed if not applicable

Where Should the Reviewer Start?

Areas for Feedback? (optional)

  • Code
  • Documentation
  • Testing
  • Codemods

Testing Manually

Screenshots or GIFs (if applicable)

Thank You Gif (optional)

Copy link

cypress bot commented Apr 14, 2025

Workday/canvas-kit    Run #8687

Run Properties:  status check passed Passed #8687  •  git commit 1b19dd1234 ℹ️: Merge d5c9cb1a156569160c9b5f6765c5cf26a4dab864 into 2c842de731765e6eb61ea244fa7e...
Project Workday/canvas-kit
Branch Review mc-remove-horizontal-ff
Run status status check passed Passed #8687
Run duration 02m 54s
Commit git commit 1b19dd1234 ℹ️: Merge d5c9cb1a156569160c9b5f6765c5cf26a4dab864 into 2c842de731765e6eb61ea244fa7e...
Committer Manuel Carrera
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 21
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 936
View all changes introduced in this branch ↗︎
UI Coverage  21.35%
  Untested elements 1523  
  Tested elements 411  
Accessibility  99.28%
  Failed rules  6 critical   5 serious   0 moderate   2 minor
  Failed elements 98  

@mannycarrera4 mannycarrera4 marked this pull request as draft April 14, 2025 22:01
Copy link
Member

@alanbsmith alanbsmith left a comment

Choose a reason for hiding this comment

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

Left a question, but this looks good. Thanks, @mannycarrera4!

@mannycarrera4 mannycarrera4 marked this pull request as ready for review April 16, 2025 16:32
@mannycarrera4 mannycarrera4 added the ready for review Code is ready for review label Apr 16, 2025
@mannycarrera4 mannycarrera4 added automerge and removed ready for review Code is ready for review labels Apr 16, 2025
@alanbsmith alanbsmith merged commit 3b6b7fb into Workday:prerelease/major Apr 16, 2025
19 of 20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants