Skip to content

refactor tagcloud to use tabify responseHandler#14266

Merged
ppisljar merged 2 commits into
elastic:masterfrom
ppisljar:ref/tagcloud
Oct 6, 2017
Merged

refactor tagcloud to use tabify responseHandler#14266
ppisljar merged 2 commits into
elastic:masterfrom
ppisljar:ref/tagcloud

Conversation

@ppisljar
Copy link
Copy Markdown
Contributor

@ppisljar ppisljar commented Oct 3, 2017

tagcloud should use tabify responseHandler to make it easy to reuse.

@ppisljar ppisljar added Feature:Visualizations Generic visualization features (in case no more specific feature label is available) review v6.0.0 v6.1.0 v7.0.0 labels Oct 3, 2017
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

this is still messy ....

ideally filter event would take the aggConfig and value (instead of using aggConfigResult)
will address this in separate PR

Copy link
Copy Markdown
Contributor

@nreese nreese left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@thomasneirynck thomasneirynck left a comment

Choose a reason for hiding this comment

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

image

:)

@ppisljar ppisljar merged commit 8cde5ef into elastic:master Oct 6, 2017
ppisljar added a commit to ppisljar/kibana that referenced this pull request Oct 6, 2017
* refactor tagcloud to use tabify responseHandler

* fixing issue with formatted word tag positioning
ppisljar added a commit to ppisljar/kibana that referenced this pull request Oct 6, 2017
* refactor tagcloud to use tabify responseHandler

* fixing issue with formatted word tag positioning
ppisljar added a commit that referenced this pull request Oct 8, 2017
* refactor tagcloud to use tabify responseHandler

* fixing issue with formatted word tag positioning
ppisljar added a commit that referenced this pull request Oct 8, 2017
* refactor tagcloud to use tabify responseHandler

* fixing issue with formatted word tag positioning
patrykkopycinski pushed a commit to patrykkopycinski/kibana that referenced this pull request May 6, 2026
* refactor tagcloud to use tabify responseHandler

* fixing issue with formatted word tag positioning
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) review v6.0.0 v6.1.0 v7.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants