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.

Sign up for GitHub

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

Jump to bottom

Tag cloud options work without applying #13298

Closed
bhavyarm opened this issue Aug 2, 2017 · 2 comments
Closed

Tag cloud options work without applying #13298

bhavyarm opened this issue Aug 2, 2017 · 2 comments
Labels
discuss Feature:Tagcloud Tag cloud visualization feature Feature:Visualizations Generic visualization features (in case no more specific feature label is available)

Comments

@bhavyarm
Copy link
Contributor

bhavyarm commented Aug 2, 2017

Kibana version: 6.0.0-beta1 BC5

Elasticsearch version: 6.0.0 - beta1 BC5

Server OS version: darwin_x86_64

Browser version: chrome latest

Browser OS version: OS X

Original install method (e.g. download page, yum, from source, etc.): from staging

Description of the problem including expected versus actual behavior: As observed on 6.0.0-beta1 BC5, if user makes changes in tag cloud - they apply without having to click apply(go button). This is the follow-up visualization refactoring: #12930. This is different to how other visualization types are handling it. Logging it here so that @elastic/kibana-visualizations can close it depending on how they move forward.

tag_cloud_np_play
@bhavyarm bhavyarm added Feature:Tagcloud Tag cloud visualization feature 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 Aug 2, 2017
@thomasneirynck thomasneirynck added discuss and removed bug Fixes for quality problems that affect the customer experience labels Aug 2, 2017
@ppisljar
Copy link
Member

ppisljar commented Aug 3, 2017

this was intentional, as tag cloud is the simplest vis we implemented it there, but we want to move forward in this direction for all charts.

@bhavyarm
Copy link
Contributor Author

bhavyarm commented Aug 3, 2017

Great. Thanks for letting me know @ppisljar. I will look at docs and roadmap. Thanks!

@ppisljar ppisljar mentioned this issue Sep 6, 2017
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
discuss Feature:Tagcloud Tag cloud visualization feature Feature:Visualizations Generic visualization features (in case no more specific feature label is available)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@thomasneirynck @bhavyarm @ppisljar