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-191] Form Pill Kit Typescript Conversion #1921

Merged

Conversation

dmcindoe
Copy link
Contributor

@dmcindoe dmcindoe commented May 23, 2022

React Kit Coverage

Screens

Screen Shot 2022-05-23 at 8 58 48 AM

Breaking Changes

[No Just converting existing code into Typescript. Should function the same]

Runway Ticket URL

[INSERT URL]](https://nitro.powerhrg.com/runway/backlog_items/PLAY-191)

How to test this

Page in Playbook, and prop should function as normal

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

dmcindoe and others added 11 commits May 3, 2022 14:27
* 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>
@dmcindoe dmcindoe self-assigned this May 23, 2022
@dmcindoe dmcindoe requested a review from a team as a code owner May 23, 2022 13:00
@dmcindoe dmcindoe added the milano 20 MAX - Deploy this PR to a review environment via Milano label May 23, 2022
@thestephenmarshall thestephenmarshall removed the milano 20 MAX - Deploy this PR to a review environment via Milano label May 23, 2022
@thestephenmarshall thestephenmarshall added enhancement New Features, Props, & Variants (USED IN CHANGELOG) Ready for Release merged to master, ready for a versioned released labels May 23, 2022
@thestephenmarshall thestephenmarshall removed the enhancement New Features, Props, & Variants (USED IN CHANGELOG) label Jun 6, 2022
@thestephenmarshall thestephenmarshall merged commit 2483e15 into master Jun 6, 2022
@thestephenmarshall thestephenmarshall deleted the Play-191-Form-Pill-Kit-Typescript-Conversion branch June 6, 2022 20:19
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.

None yet

3 participants