[Launch Instance Fix] Fix donut chart title data and standardize design
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenStack Dashboard (Horizon) |
Fix Released
|
Low
|
Brian Tully |
Bug Description
Currently, each step passes in settings for the donut charts. These can be standardized in the chart itself, and overrides removed from each step. In addition we can add constants to the charts for label names (so that they may be translated) and css classes. These can then be leveraged for validation UX. In addition, there is a bug where the chart title shows (# Max), however this # is not really the Max limit and instead is a sum of usage, added and available. We should add a maxLimit property to the chartData object that when passed will ensure it stays intact and therefore can be used for validation. If no maxLimit is specified, the current chart logic can be used to sum the usage, added and available items, but we should change the title to show (# Total) instead of (# Max)
Changed in horizon: | |
assignee: | nobody → Brian Tully (brian-tully) |
summary: |
- [Launch Instance Fix] Make donut chart styles common for all steps + [Launch Instance Fix] Fix donut chart title data and standardize design |
description: | updated |
tags: | added: ng-subteam |
Changed in horizon: | |
assignee: | Brian Tully (brian-tully) → Kelly Domico (kelly-domico) |
Changed in horizon: | |
assignee: | Kelly Domico (kelly-domico) → Brian Tully (brian-tully) |
Changed in horizon: | |
assignee: | Brian Tully (brian-tully) → Kelly Domico (kelly-domico) |
Changed in horizon: | |
assignee: | Kelly Domico (kelly-domico) → Brian Tully (brian-tully-k) |
Changed in horizon: | |
importance: | Undecided → Low |
Changed in horizon: | |
assignee: | Brian Tully (brian-tully-k) → Brian Tully (brian-tully) |
Changed in horizon: | |
milestone: | none → liberty-1 |
Changed in horizon: | |
status: | Fix Committed → Fix Released |
Changed in horizon: | |
milestone: | liberty-1 → 8.0.0 |
Fix proposed to branch: master /review. openstack. org/167377
Review: https:/