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-450] Charts/Graphs kits legend position #2171

Merged
merged 15 commits into from Nov 14, 2022

Conversation

Israel-Molestina
Copy link
Contributor

@Israel-Molestina Israel-Molestina commented Nov 3, 2022

Screens

Old

Screen Shot 2022-11-03 at 2 59 51 PM

New

Screen Shot 2022-11-04 at 12 06 25 PM

Screen Shot 2022-11-04 at 12 06 54 PM

Breaking Changes

NO

Runway Ticket URL

Runway TIcket

How to test this

You can test this by using each new prop and making sure each value has the intended outcome

Also please look at the new doc examples for both rails and react in the bar graph, circle chart and line graph kits to make sure they are easy to read and digest.

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

@Israel-Molestina Israel-Molestina added enhancement New Features, Props, & Variants (USED IN CHANGELOG) milano 20 MAX - Deploy this PR to a review environment via Milano labels Nov 3, 2022
@Israel-Molestina Israel-Molestina requested a review from a team as a code owner November 3, 2022 19:03
@Israel-Molestina Israel-Molestina self-assigned this Nov 3, 2022
@app-milano app-milano bot temporarily deployed to pr2171 November 3, 2022 19:12 Inactive
@app-milano app-milano bot temporarily deployed to pr2171 November 4, 2022 15:09 Inactive
@app-milano app-milano bot temporarily deployed to pr2171 November 4, 2022 15:46 Inactive
@Israel-Molestina Israel-Molestina added milano 20 MAX - Deploy this PR to a review environment via Milano and removed milano 20 MAX - Deploy this PR to a review environment via Milano labels Nov 4, 2022
@app-milano app-milano bot temporarily deployed to pr2171 November 4, 2022 16:31 Inactive
Copy link
Contributor

@nidaqg nidaqg left a comment

Choose a reason for hiding this comment

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

This looks really good Israel! I like that it's also spelled out what the type is so there's no confusion on what can be passed to the prop. We need this in more of our docs over all

…being passed correctly and update the doc examples for legend position
…eing passed correctly and update the doc examples for legend position
@jasperfurniss jasperfurniss added the Ready for Release merged to master, ready for a versioned released label Nov 14, 2022
@nidaqg nidaqg merged commit 35ea2fd into master Nov 14, 2022
@nidaqg nidaqg deleted the PLAY-450-charts-legend-position branch November 14, 2022 19:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New Features, Props, & Variants (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.

None yet

3 participants