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-1178] Fixed Styling when Select Kit is used within a Flex Kit #3414

Merged
merged 1 commit into from
May 20, 2024

Conversation

jasperfurniss
Copy link
Contributor

@jasperfurniss jasperfurniss commented May 16, 2024

What does this PR do? A clear and concise description with your runway ticket url.
This allows the dev more fine-grained control over the elements inside of our Select kit, and enables them to move things around using our global spacing props-- which in turn, fixes a bug where the Select kit is broken when nested inside of a Flex kit.

Screenshots: Screenshots to visualize your addition/change
Screenshot 2024-05-17 at 9 25 11 AM

Screenshot 2024-05-17 at 9 25 25 AM

How to test? Steps to confirm the desired behavior:
https://codesandbox.io/p/sandbox/funny-booth-rqd9dq

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.

@jasperfurniss jasperfurniss requested review from a team as code owners May 16, 2024 16:10
@jasperfurniss jasperfurniss self-assigned this May 16, 2024
@jasperfurniss jasperfurniss added milano 20 MAX - Deploy this PR to a review environment via Milano alpha labels May 16, 2024
Copy link

🎉 Congratulations on creating an Alpha Version!

Your Alpha for Ruby Gems is 13.27.0.pre.alpha.PLAY11782916

Your Alpha for NPM is 13.27.0-alpha.PLAY11782916

@jasperfurniss jasperfurniss changed the title [PLAY-1178] Added selectProps & iconProps to Select Kit [PLAY-1178] Fixed Styling when Select Kit is used within a Flex Kit May 17, 2024
@jasperfurniss jasperfurniss added alpha and removed alpha labels May 17, 2024
Copy link

🎉 Congratulations on creating an Alpha Version!

Your Alpha for Ruby Gems is 13.27.0.pre.alpha.PLAY11782918

Your Alpha for NPM is 13.27.0-alpha.PLAY11782918

@jasperfurniss jasperfurniss added this pull request to the merge queue May 20, 2024
Merged via the queue into master with commit de61e50 May 20, 2024
5 checks passed
@jasperfurniss jasperfurniss deleted the PLAY-1178 branch May 20, 2024 17:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
alpha milano 20 MAX - Deploy this PR to a review environment via Milano
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants