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

Changing legacy visualization type doesn't update icon #11331

Closed
cjcenizal opened this issue Apr 19, 2017 · 6 comments · Fixed by #119377
Closed

Changing legacy visualization type doesn't update icon #11331

cjcenizal opened this issue Apr 19, 2017 · 6 comments · Fixed by #119377
Assignees
Labels
bug Fixes for quality problems that affect the customer experience Feature:Visualizations Generic visualization features (in case no more specific feature label is available) impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. Team:Visualizations Visualization editors, elastic-charts and infrastructure

Comments

@cjcenizal
Copy link
Contributor

cjcenizal commented Apr 19, 2017

This may be related to #11242. This issue exists in 6.0.0.

If you create an area chart visualization and then change the type to "bar" under the Options tab, the "type" metadata for this visualization doesn't get updated, so an incorrect icon for the visualization is displayed on the landing page.

CC @bhavyarm

@cjcenizal cjcenizal added Feature:Visualizations Generic visualization features (in case no more specific feature label is available) bug Fixes for quality problems that affect the customer experience labels Apr 19, 2017
@ppisljar ppisljar self-assigned this Apr 26, 2017
@ppisljar
Copy link
Member

ppisljar commented Sep 5, 2017

@snide should we have an icon for a mixed chart type (line + bar + area) or should we default to line in those mixed scenarios ?

@tsullivan
Copy link
Member

I ran into this problem today and did some digging:

This seems to happen because there are different sources of the visualization type metadata:

There can be multiple metrics on a chart and they can have different types, which the "mixed" chart type that @ppisljar is referring to.

@ppisljar
Copy link
Member

@tsullivan yeah icon changing is not implemented at all.
if we would implement it, on save we would check all the seriesParams[i].type and set the appropriate type` on the visState. We would then need this mixed option to represent charts with multiple series types.

@cjcenizal
Copy link
Contributor Author

@timroes timroes added the Team:Visualizations Visualization editors, elastic-charts and infrastructure label Sep 16, 2018
@ppisljar ppisljar removed their assignment Feb 11, 2019
@thomai-d
Copy link

This bug is still present in 7.1.0

I created the chart as AreaChart initially, but changed it to a LineChart later:

grafik

grafik

@prabhakartalari
Copy link

May i know is there any solution. I am using Elastic 7.6.2 Version and could see the issue still exist.

@stratoula stratoula added the impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. label Jul 26, 2021
@flash1293 flash1293 added impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. and removed impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. labels Oct 11, 2021
@drewdaemon drewdaemon self-assigned this Nov 22, 2021
@drewdaemon drewdaemon changed the title Visualization "type" isn't updated when the type is changed under the options tab Changing legacy visualization type doesn't update icon Nov 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience Feature:Visualizations Generic visualization features (in case no more specific feature label is available) impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. Team:Visualizations Visualization editors, elastic-charts and infrastructure
Projects
None yet
Development

Successfully merging a pull request may close this issue.

9 participants