Aerogear 7693 - Create Mobile Client Service Chart #12
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I did not test it locally. However, the code shows 100% fine.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@wojta that was part of Johns work, this PR is just to add the donut chart, I left a note in the description that its best wait for us to get johns work merged first and then refactor/rebase this as needed. |
ui/src/MobileClientServiceChart.js
Outdated
import './styles/MobileClientServiceChart.css' | ||
|
||
class MobileClientServiceChart extends Component { | ||
viewAllServices(){ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not really part of this component. We can move it when we integrate all the components though
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Could we add the number of bound/unbound services to the text next to the pie chart like the flat has.
7cccdd8
to
8d86359
Compare
8d86359
to
277cee1
Compare
Motivation
JIRA - https://issues.jboss.org/browse/AEROGEAR-7693
Description
Based on top of @sedroche fork, this change adds a Mobile Service Chart Component. The component expects two props, the number of unbound service and bound services.
Screenshot
NOTE : gif does not capture the mouse pointer correctly
Note
Might be best to wait on @sedroche to be merged first and rebase/refactor this PR as needs be, then.