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

Ensure tooltips are labeled with percentage values #12746

Closed

Conversation

thomasneirynck
Copy link
Contributor

@thomasneirynck thomasneirynck commented Jul 11, 2017

Release note: Tooltips now correcly show the percentage value if a Y-axis indicates percentage.

Closes #12391.

@thomasneirynck thomasneirynck added Feature:Visualizations Generic visualization features (in case no more specific feature label is available) release_note:fix v5.6.0 v6.0.0-beta1 labels Jul 11, 2017
@thomasneirynck thomasneirynck requested a review from ppisljar July 11, 2017 03:05
Copy link
Member

@ppisljar ppisljar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i like the solution for dispatch, but i don't think agg_configs should know anything about vislib or options like percentage.

return pre += this.type.makeLabel(this);
};

AggConfig.prototype.isPercentage = function () {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

actually this is creating couplings that shouldn''t be there ... agg_config shouldn't have to know anything about vislib.
also it probably shouldn't have to know anything about percentage options and such.

i think vislib should take care of this when rendering the tooltip.

@alexfrancoeur
Copy link

Not seeing the percentage with the following configuration in makelogs

Metric: Count
X-axis: Date histogram
Split series: Terms, host.raw
LeftAxis1: Percentage mode

screen shot 2017-07-14 at 12 36 51 pm

screen shot 2017-07-14 at 12 37 03 pm

cc: @thomasneirynck

@thomasneirynck thomasneirynck force-pushed the fix/percentageFormatting branch from 64c8ca8 to c034a05 Compare July 24, 2017 20:56
@ppisljar ppisljar mentioned this pull request Jul 26, 2017
1 task
@thomasneirynck
Copy link
Contributor Author

Closing this in favor of #13212.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Visualizations Generic visualization features (in case no more specific feature label is available) release_note:fix v5.6.0 v6.0.0-beta1 v6.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants