Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[PLAY-698] responsive Spacing global prop #2563

Merged
merged 21 commits into from
May 26, 2023

Conversation

nickamantia
Copy link
Contributor

@nickamantia nickamantia commented May 16, 2023

What does this PR do? A clear and concise description with your runway ticket url.
PLAY-698

This PR:

  • Adds new spacing props auto,initial, and inherit
  • Updates the Spacing global props so they are responsive
  • Updates the docs to show an example of new responsive functionality and props and has an outbound link to a Codesandbox.

Screenshots
Screen Shot 2023-05-24 at 11 26 42 AM
:
Screenshots to visualize your addition/change

Rails:

Screen.Recording.2023-05-23.at.02.38.59.PM.mp4

React:

Screen.Recording.2023-05-23.at.02.44.31.PM.mp4

Updated Global Props table with auto,initial, and inherit
Screen Shot 2023-05-23 at 1 38 20 PM

How to test? Steps to confirm the desired behavior:

  1. Go to any kit
  2. Add responsive spacing prop
  • React Example: padding={{ xs: "xxs", sm: "xs", md: "xl", lg: "md", xl: "none" }}

  • Rails Example: padding: { xs: "xxs", sm: "xs", md: "xl", lg: "md", xl: "none" }

  1. change size of browser and see if the spacing changes.

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.

@nickamantia nickamantia self-assigned this May 16, 2023
@jasperfurniss jasperfurniss added the milano 20 MAX - Deploy this PR to a review environment via Milano label May 18, 2023
@app-milano app-milano bot temporarily deployed to pr2563 May 18, 2023 20:20 Inactive
@app-milano app-milano bot temporarily deployed to pr2563 May 19, 2023 16:06 Inactive
@github-actions
Copy link

🎉 Congratulations on creating an Alpha Version!

Your Alpha for Ruby Gems is 12.22.0.pre.alpha.play698responsivespacingglobalprop690

Your Alpha for NPM is 12.22.0-alpha.play698responsivespacingglobalprop690

@app-milano app-milano bot temporarily deployed to pr2563 May 22, 2023 21:29 Inactive
@app-milano app-milano bot temporarily deployed to pr2563 May 23, 2023 14:56 Inactive
@nickamantia nickamantia added alpha and removed alpha labels May 23, 2023
@github-actions
Copy link

🎉 Congratulations on creating an Alpha Version!

Your Alpha for Ruby Gems is 12.23.0.pre.alpha.play698responsivespacingglobalprop701

Your Alpha for NPM is 12.23.0-alpha.play698responsivespacingglobalprop701

@nickamantia nickamantia added the improvement This is used when your PR contains library upgrades or doc/site improvements. (USED IN CHANGELOG)) label May 23, 2023
@app-milano app-milano bot temporarily deployed to pr2563 May 23, 2023 18:03 Inactive
@nickamantia nickamantia marked this pull request as ready for review May 23, 2023 18:51
@nickamantia nickamantia requested a review from a team as a code owner May 23, 2023 18:51
@app-milano app-milano bot temporarily deployed to pr2563 May 23, 2023 19:14 Inactive
@nickamantia nickamantia added alpha and removed alpha labels May 24, 2023
@github-actions
Copy link

🎉 Congratulations on creating an Alpha Version!

Your Alpha for Ruby Gems is 12.23.0.pre.alpha.play698responsivespacingglobalprop703

Your Alpha for NPM is 12.23.0-alpha.play698responsivespacingglobalprop703

@app-milano app-milano bot temporarily deployed to pr2563 May 24, 2023 15:39 Inactive
@jasperfurniss jasperfurniss added the Ready for Release merged to master, ready for a versioned released label May 25, 2023
@jasperfurniss jasperfurniss added this pull request to the merge queue May 26, 2023
Merged via the queue into master with commit d1d2b28 May 26, 2023
@jasperfurniss jasperfurniss deleted the play-698-responsive-spacing-global-prop branch May 26, 2023 12:43
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
alpha improvement This is used when your PR contains library upgrades or doc/site improvements. (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 release 2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants