Project

General

Profile

Bug #13424

Add Patternfly donut charts support

Added by Daniel Lobato Garcia about 4 years ago. Updated over 1 year ago.

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

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 compatibilityNew2016-09-12
Related to Foreman Remote Execution - Bug #16516: Explicitly require chart.js for 1.14 compatibilityNew2016-09-12
Related to Foreman - Bug #10201: Statistics page behavior when looking at an empty org looks brokenResolved2015-04-17
Related to Foreman - Bug #17569: React Chart Component ErrorClosed2016-12-04

Associated revisions

Revision 659d49a1 (diff)
Added by Gail Steiger over 3 years ago

Fixes #13424 - c3 patternfly react implementation

Revision 3f9c4075 (diff)
Added by Dominic Cleal over 3 years ago

refs #13424 - add wait_for_ajax to statistics page load test

Revision 9ff20c7f (diff)
Added by Dominic Cleal over 3 years ago

refs #13424 - add c3, react npm dependencies

Revision c5788de1 (diff)
Added by Dominic Cleal over 3 years ago

refs #13424 - update patternfly-sass

History

#1 Updated by The Foreman Bot about 4 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 about 4 years ago

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

#3 Updated by Dominic Cleal about 4 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 almost 4 years ago

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

#5 Updated by Daniel Lobato Garcia over 3 years ago

  • Target version set to 1.7.1

#6 Updated by Daniel Lobato Garcia over 3 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 over 3 years ago

  • Target version changed from 1.7.1 to 1.6.2

#8 Updated by Marek Hulán over 3 years ago

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

#9 Updated by Marek Hulán over 3 years ago

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

#10 Updated by Daniel Lobato Garcia over 3 years ago

  • Target version changed from 1.6.2 to 1.5.2

#11 Updated by Gail Steiger over 3 years ago

  • Status changed from Ready For Testing to Closed

#12 Updated by Dominic Cleal over 3 years ago

  • Legacy Backlogs Release (now unused) set to 189

#13 Updated by The Foreman Bot over 3 years ago

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

#14 Updated by Marek Hulán over 3 years ago

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

#15 Updated by Dominic Cleal over 3 years ago

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

Also available in: Atom PDF