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 300] - Fix Global Responsive Props and Cleanup #2052

Merged
merged 2 commits into from
Sep 10, 2022

Conversation

nidaqg
Copy link
Contributor

@nidaqg nidaqg commented Sep 6, 2022

Screens

  • Flex direction global prop showing up in DOM
    Screen Shot 2022-09-06 at 9 36 10 AM

  • Flex direction global prop in action (column for smaller screen size, row for larger size)
    Screen Shot 2022-09-06 at 9 36 18 AM
    Screen Shot 2022-09-06 at 9 36 26 AM

  • justifyContent global prop showing up in DOM
    Screen Shot 2022-09-06 at 10 25 32 AM

  • justifyContent set to 'end for xs-md screen sizes for testing, working well
    Screen Shot 2022-09-06 at 10 25 40 AM

  • Moved partials for flexbox global props to their own folder for better organization

Screen Shot 2022-09-06 at 10 46 31 AM

Breaking Changes

No breaking changes, fixed broken props

Runway Ticket URL

Runway Story

How to test this

[INSERT TESTING DETAILS]

Checklist:

  • LABELS Add a label: enhancement, bug, improvement, new kit, deprecated, or breaking. See Changelog & Labels for details.
  • DEPLOY Please add the Milano label when you are ready for a review.
  • SCREENSHOT Please add a screen shot or two.
  • SPECS Please cover your changes with specs.
  • READ DOCS Please make sure you have read and understand the Playbook Release Process

@nidaqg nidaqg added enhancement New Features, Props, & Variants (USED IN CHANGELOG) milano 20 MAX - Deploy this PR to a review environment via Milano labels Sep 6, 2022
@nidaqg nidaqg requested a review from a team as a code owner September 6, 2022 14:57
@jasperfurniss jasperfurniss added the Ready for Release merged to master, ready for a versioned released label Sep 9, 2022
@jasperfurniss jasperfurniss merged commit 939fa01 into master Sep 10, 2022
@jasperfurniss jasperfurniss deleted the PLAY-300-fix-global-responsive-props branch September 10, 2022 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New Features, Props, & Variants (USED IN CHANGELOG) milano 20 MAX - Deploy this PR to a review environment via Milano Ready for Release merged to master, ready for a versioned released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants