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

Visualization editor restyle #3616

Merged
merged 3 commits into from
Apr 20, 2015

Conversation

rashidkpc
Copy link
Contributor

Closes #3348

This was largely accelerated as a response to #3589 which adds a large number of important view options. This moves view options into their own section, decluttering the side bar and allowing for more, easier to find, view options.

screen shot 2015-04-16 at 11 38 26 am

@spalger
Copy link
Contributor

spalger commented Apr 16, 2015

I like it! Wish the bar above the metrics was a bit shorter, but oh well.

@spalger spalger assigned rashidkpc and unassigned spalger Apr 16, 2015
@rashidkpc rashidkpc assigned spalger and unassigned rashidkpc Apr 16, 2015
@spalger
Copy link
Contributor

spalger commented Apr 17, 2015

Can we make this fit a bit better on mobile?

image

@rashidkpc
Copy link
Contributor Author

Now with improved mobiles.

image

@spalger
Copy link
Contributor

spalger commented Apr 20, 2015

LGTM

spalger added a commit that referenced this pull request Apr 20, 2015
@spalger spalger merged commit e538ed6 into elastic:master Apr 20, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Move view options to separate panel
2 participants