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-346] Update pb_chart plugin to resize gauge kit on load #2089

Merged
merged 5 commits into from
Sep 30, 2022

Conversation

Israel-Molestina
Copy link
Contributor

@Israel-Molestina Israel-Molestina commented Sep 23, 2022

Screens

Screen Shot 2022-09-23 at 3 05 59 PM

Breaking Changes

Potentially any Gauge kits that are nested within a FlexItem with the fixedSize prop. If the fixedSize is less than 150px then you would need to add the prop: height="100%" to the Gauge kit.

Runway Ticket URL

Runway URL

How to test this

Check if all the Gauge Kit examples are properly sized on page load.

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

…art.js file, Created a doc example showing how a gauge can work nested in muliple other flex and card kits, added align=center to the flex component in the sizing.js example as well as height=100% to the last example in the sizing.js card so it renders the graph properly
@Israel-Molestina Israel-Molestina added bug Fixes to issues discovered in Playbook (USED IN CHANGELOG) milano 20 MAX - Deploy this PR to a review environment via Milano labels Sep 23, 2022
@Israel-Molestina Israel-Molestina self-assigned this Sep 23, 2022
@Israel-Molestina Israel-Molestina requested a review from a team as a code owner September 23, 2022 18:00
@app-milano app-milano bot temporarily deployed to pr2089 September 23, 2022 18:13 Inactive
@app-milano app-milano bot temporarily deployed to pr2089 September 23, 2022 18:39 Inactive
@jasperfurniss
Copy link
Contributor

Screens

Screen Shot 2022-09-23 at 1 57 05 PM

Breaking Changes

NO

Runway Ticket URL

Runway URL

How to test this

Check if all the Gauge Kit examples are properly sized on page load.

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 Can you update with a fresh screenshot?

@Israel-Molestina Israel-Molestina added Ready for Release merged to master, ready for a versioned released and removed Ready for Release merged to master, ready for a versioned released labels Sep 30, 2022
@jasperfurniss jasperfurniss merged commit 282d3b1 into master Sep 30, 2022
@jasperfurniss jasperfurniss deleted the PLAY-346-Gauge-Kit-Resize branch September 30, 2022 17:15
@jasperfurniss jasperfurniss added the Ready for Release merged to master, ready for a versioned released label Sep 30, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Fixes to issues discovered in Playbook (USED IN CHANGELOG) milano 20 MAX - Deploy this PR to a review environment via Milano needs alpha testing 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