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-182] Update Success And Error Props #1900

Merged
merged 16 commits into from
Jun 6, 2022

Conversation

CooperSawyer1
Copy link
Contributor

@CooperSawyer1 CooperSawyer1 commented May 12, 2022

Screens

Rails in light mode:
Screen Shot 2022-05-17 at 10 57 11 AM

Rails in dark mode:
Screen Shot 2022-05-17 at 10 59 37 AM

React light mode:
Screen Shot 2022-05-17 at 10 57 17 AM

React in dark mode:
Screen Shot 2022-05-17 at 10 58 16 AM

Breaking Changes

Deprecated the status prop but everything should still work correctly.

Runway Ticket URL

Play-182

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 10 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>
…om:powerhome/playbook into Play-182-update-add-succes-and-error-props
@CooperSawyer1 CooperSawyer1 self-assigned this May 12, 2022
@CooperSawyer1 CooperSawyer1 requested a review from a team as a code owner May 12, 2022 21:28
@thestephenmarshall thestephenmarshall force-pushed the master branch 2 times, most recently from 4755d3c to 9a62f6e Compare May 13, 2022 13:30
@CooperSawyer1 CooperSawyer1 added enhancement New Features, Props, & Variants (USED IN CHANGELOG) and removed wip labels May 17, 2022
@CooperSawyer1 CooperSawyer1 changed the title [Play-182] update success and error props [Play-182] Update Success And Error Props May 17, 2022
@thestephenmarshall thestephenmarshall added the Ready for Release merged to master, ready for a versioned released label May 23, 2022
@thestephenmarshall thestephenmarshall merged commit 778c08b into master Jun 6, 2022
@thestephenmarshall thestephenmarshall deleted the Play-182-update-success-and-error-props branch June 6, 2022 20:21
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New Features, Props, & Variants (USED IN CHANGELOG) Ready for Release merged to master, ready for a versioned released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants