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

Create React Line Height Partial #1892

Merged

Conversation

THElvan
Copy link
Member

@THElvan THElvan commented May 11, 2022

React Kit Coverage

Screens

Screen.Recording.2022-05-11.at.2.20.59.PM.mov

Breaking Changes

No. No logic changes and just added it to the visual_guidelines_react.

Runway Ticket URL

PLAY-167

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

thestephenmarshall and others added 5 commits May 5, 2022 10:18
* Removed flow and Renamed file

* Updated Types

* Removed JSX file extentions from imports

* Fix syntax error

* Fix tests

Co-authored-by: Erica Winne <erica.winne@powerhrg.com>
* Started Converting

* Rename file to TSX and Updated types

* Added/Joined & GlobalProps

* Merge Master

* Changed back default blank string

Co-authored-by: Erica Winne <erica.winne@powerhrg.com>
…1849)

* Rendered rough example in browser

* Section separator between props and values

* Added pills and continued to change styling

* Fixed width of cards

* Fixed flexitem alignment

* Added title and body kits

* Fixed more styling

* Changed out background kit to fix border radius

* Fixed border radius

* Fixed border radius on other templates

* Updated caption card

* Fixed title margins

* Corrected camelcase in pills

* Three templates for visual guidelines

* Fixed syntax error

* Fixed syntax again

* Testing client-side code example colorization

* Use Rouge to format code examples

* Template Complete

Co-authored-by: Stephen Marshall <smarshall1980@gmail.com>
@THElvan THElvan requested a review from a team as a code owner May 11, 2022 18:30
@THElvan THElvan self-assigned this May 11, 2022
@THElvan THElvan added the milano 20 MAX - Deploy this PR to a review environment via Milano label May 11, 2022
Copy link
Member

@jasoncypret jasoncypret left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀

Copy link
Contributor

@thestephenmarshall thestephenmarshall left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mint!

@app-milano app-milano bot temporarily deployed to pr1892 May 11, 2022 20:06 Inactive
Copy link
Contributor

@thestephenmarshall thestephenmarshall left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This one also needs master merged in.

@thestephenmarshall thestephenmarshall changed the title [PLAY-167] Create React Line Height Partial Create React Line Height Partial May 12, 2022
@thestephenmarshall thestephenmarshall added the Ready for Release merged to master, ready for a versioned released label May 12, 2022
@THElvan THElvan removed the milano 20 MAX - Deploy this PR to a review environment via Milano label May 13, 2022
@thestephenmarshall thestephenmarshall merged commit c99d2af into master Jun 7, 2022
@thestephenmarshall thestephenmarshall deleted the PLAY-167-switch-line-height-partial-to-react branch June 7, 2022 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready for Release merged to master, ready for a versioned released typescript
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants