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

Template vistype and Table Vis #1763

Closed
wants to merge 90 commits into from
Closed

Conversation

spalger
Copy link
Contributor

@spalger spalger commented Oct 28, 2014

Spencer Alger added 30 commits October 22, 2014 08:56
…ling of hierarchical, and spec'd out renderbots
…ls, use a controller, and provide UI for page size
Conflicts:
	src/kibana/plugins/vis_types/index.js
	test/unit/index.html
@spalger spalger added WIP and removed review labels Nov 4, 2014
Spencer Alger and others added 3 commits November 4, 2014 11:41
- wrap all vis types tests
- wrap vislib tests
- add basic renderbot and vislib renderbot tests
@brwe
Copy link
Contributor

brwe commented Nov 4, 2014

For a term aggregation if term starts with a number it is centered:

image

Spencer Alger added 5 commits November 4, 2014 13:58
@spalger
Copy link
Contributor Author

spalger commented Nov 4, 2014

@brwe yeah... that's certainly a bug. Opened #1810 to track it since it's not really caused the by PR

@rashidkpc rashidkpc removed the WIP label Nov 4, 2014
@spalger spalger added the review label Nov 4, 2014
@spalger
Copy link
Contributor Author

spalger commented Nov 4, 2014

@w33ble I've responded to all of your issues by updating the comments with a "response".

All should be addressed and fixed.

@w33ble
Copy link
Contributor

w33ble commented Nov 5, 2014

Poked through it again, LGTM. Just need to merge master.

@w33ble
Copy link
Contributor

w33ble commented Nov 5, 2014

Moving to #1783 since all the changes are merged in there

@w33ble w33ble closed this Nov 5, 2014
This was referenced Nov 6, 2014
@spalger spalger deleted the template_vistype branch December 12, 2014 21:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants