Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix stderr checkbox disablement in Metric Explorer. #1184

Merged
merged 1 commit into from
Dec 31, 2019
Merged

Conversation

jpwhite4
Copy link
Member

The "Err Bars" and "Err Labels" columns in the "Data" menu in the metric
explorer were incorrectly shown or incorrectly not shown in certian
circumstances.

a) if a chart was 'geared over' from the Usage Tab then the "Data" menu would
always show the "Err Bars" and "Err Labels" checkboxes regardless of whether
they should have been shown.

b) if data were added to the metric explorer from the "Metric Catalog" then
the "Data" menu would never show the checkboxes regardless of whether they
should have been shown.

If data are added to a chart via the "Data Series Definition" dialog then the
checkboxes are correct.

This change fixes it so the checkboxes work correctly.

Old:
image
New:
image

Old:
image
New:
image

The "Err Bars" and "Err Labels" columns in the "Data" menu in the metric
explorer were incorrectly shown or incorrectly not shown in certian
circumstances.

a) if a chart was 'geared over' from the Usage Tab then the "Data" menu would
always show the "Err Bars" and "Err Labels" checkboxes regardless of whether
they should have been shown.

b) if data were added to the metric explorer from the "Metric Catalog" then
the "Data" menu would never show the checkboxes regardless of whether they
should have been shown.

If data are added to a chart via the "Data Series Definition" dialog then the
checkboxes are correct.

This change fixes it so the checkboxes work correctly.
@jpwhite4 jpwhite4 added this to the 9.0.0 milestone Dec 12, 2019
@jpwhite4 jpwhite4 added bug Bugfixes Category:Metric Explorer Metric Explorer / Usage labels Dec 12, 2019
@jpwhite4 jpwhite4 merged commit 41be097 into xdmod9.0 Dec 31, 2019
@jpwhite4 jpwhite4 deleted the has_std_err branch December 31, 2019 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bugfixes Category:Metric Explorer Metric Explorer / Usage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants