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: Ensure width is passed to Select input #3130

Merged
merged 2 commits into from
Feb 10, 2025

Conversation

mannycarrera4
Copy link
Contributor

@mannycarrera4 mannycarrera4 commented Feb 10, 2025

Summary

Width was deconstructed from Select input props. We should remove it it from the deconstructed props to ensure correct styling when you pass with to Select.Input.

Release Category

Components


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)

@mannycarrera4 mannycarrera4 added the ready for review Code is ready for review label Feb 10, 2025
@mannycarrera4 mannycarrera4 changed the base branch from master to support February 10, 2025 18:38
Copy link

cypress bot commented Feb 10, 2025

Workday/canvas-kit    Run #8291

Run Properties:  status check passed Passed #8291  •  git commit 89b08a697e ℹ️: Merge fe962df75a49d027fc675480d7ccd322b611a10f into ab29c0d8306dfaa2692f0c9c67c2...
Project Workday/canvas-kit
Branch Review refs/pull/3130/merge
Run status status check passed Passed #8291
Run duration 05m 19s
Commit git commit 89b08a697e ℹ️: Merge fe962df75a49d027fc675480d7ccd322b611a10f into ab29c0d8306dfaa2692f0c9c67c2...
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 3
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 1006
View all changes introduced in this branch ↗︎

@alanbsmith alanbsmith merged commit 1849540 into Workday:support Feb 10, 2025
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge ready for review Code is ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants