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-317] - Visual Guidelines Improvements #2055

Merged
merged 2 commits into from
Sep 10, 2022

Conversation

nidaqg
Copy link
Contributor

@nidaqg nidaqg commented Sep 8, 2022

Screens

Spacing Props:

Screen Shot 2022-09-08 at 11 19 54 AM

Spacing Tokens on smaller screens:

Screen Shot 2022-09-08 at 11 23 47 AM

LineHeight Tokens on smaller screens:

Screen Shot 2022-09-08 at 11 20 35 AM

Positioning Tokens on smaller screens:

Screen Shot 2022-09-08 at 11 21 20 AM

Breaking Changes

No breaking changes, edits to visual guidelines react styling

Runway Ticket URL

Screenshots of what screens looked like before changes can be seen on linked runway ticket:

Runway Ticket

How to test this

Review in milano env at /visual_guidelines_react
Changes to Spacing, Positioning and Line Height examples

Make sure to review react version of visual guidelines (as specified in url above). No changes to rails version of guidelines

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 milano 20 MAX - Deploy this PR to a review environment via Milano improvement This is used when your PR contains library upgrades or doc/site improvements. (USED IN CHANGELOG)) labels Sep 8, 2022
@nidaqg nidaqg self-assigned this Sep 8, 2022
@nidaqg nidaqg requested a review from a team as a code owner September 8, 2022 15:42
@jasperfurniss jasperfurniss added the Ready for Release merged to master, ready for a versioned released label Sep 9, 2022
@jasperfurniss jasperfurniss merged commit 9d82fca into master Sep 10, 2022
@jasperfurniss jasperfurniss deleted the PLAY-317-visual-guidelines-improvements-spacing branch September 10, 2022 14:38
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants