Bug #13424

Add Patternfly donut charts support

Added by Daniel Lobato Garcia over 2 years ago. Updated 8 days ago.

Status:Closed
Priority:Normal
Assignee:Gail Steiger
Category:Web Interface
Target version:1.14.0
Difficulty: Team Backlog:
Triaged: Fixed in Releases:
Bugzilla link: Found in Releases:
Pull request:https://github.com/theforeman/foreman/pull/3931, https://github.com/theforeman/foreman/pull/3603

Description

Our donut charts do not look like they should in Patternfly. https://www.patternfly.org/patterns/donut-chart/ uses c3, so we can do that too.
Eventually I want to remove flot_pies too, but I have to yet figure out how to auto refresh donut charts.
Adding this library also will allow us to change our charts in Host#show to the newer style. https://www.patternfly.org/widgets/#basic-charts


Related issues

Related to OpenSCAP - Bug #16517: Explicitly require chart.js for 1.14 compatibility New 09/12/2016
Related to Foreman Remote Execution - Bug #16516: Explicitly require chart.js for 1.14 compatibility New 09/12/2016
Related to Foreman - Bug #10201: Statistics page behavior when looking at an empty org loo... Resolved 04/17/2015
Related to Foreman - Bug #17569: React Chart Component Error Closed 12/04/2016

Associated revisions

Revision 659d49a1
Added by Gail Steiger almost 2 years ago

Fixes #13424 - c3 patternfly react implementation

Revision 3f9c4075
Added by Dominic Cleal almost 2 years ago

refs #13424 - add wait_for_ajax to statistics page load test

Revision 9ff20c7f
Added by Dominic Cleal almost 2 years ago

refs #13424 - add c3, react npm dependencies

Revision c5788de1
Added by Dominic Cleal almost 2 years ago

refs #13424 - update patternfly-sass

History

#1 Updated by The Foreman Bot over 2 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Daniel Lobato Garcia
  • Pull request https://github.com/theforeman/foreman/pull/3128 added

#2 Updated by Daniel Lobato Garcia over 2 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#3 Updated by Dominic Cleal over 2 years ago

  • Status changed from Closed to Ready For Testing

PR was accidentally merged to develop in revision f4b2a116, reverted in 743c84c8.

#4 Updated by The Foreman Bot about 2 years ago

  • Pull request https://github.com/theforeman/foreman/pull/3603 added

#5 Updated by Daniel Lobato Garcia almost 2 years ago

  • Target version set to 1.7.1

#6 Updated by Daniel Lobato Garcia almost 2 years ago

  • Assignee changed from Daniel Lobato Garcia to Gail Steiger
  • Pull request deleted (https://github.com/theforeman/foreman/pull/3128)

#7 Updated by Daniel Lobato Garcia almost 2 years ago

  • Target version changed from 1.7.1 to 1.6.2

#8 Updated by Marek Hulán almost 2 years ago

  • Related to Bug #16517: Explicitly require chart.js for 1.14 compatibility added

#9 Updated by Marek Hulán almost 2 years ago

  • Related to Bug #16516: Explicitly require chart.js for 1.14 compatibility added

#10 Updated by Daniel Lobato Garcia almost 2 years ago

  • Target version changed from 1.6.2 to 1.5.2

#11 Updated by Gail Steiger almost 2 years ago

  • Status changed from Ready For Testing to Closed

#12 Updated by Dominic Cleal almost 2 years ago

  • Legacy Backlogs Release (now unused) set to 189

#13 Updated by The Foreman Bot almost 2 years ago

  • Pull request https://github.com/theforeman/foreman/pull/3931 added

#14 Updated by Marek Hulán over 1 year ago

  • Related to Bug #10201: Statistics page behavior when looking at an empty org looks broken added

#15 Updated by Dominic Cleal over 1 year ago

  • Related to Bug #17569: React Chart Component Error added

Also available in: Atom PDF