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

Tighten up spacing, move drag handle, move spy toggle #6247

Merged
merged 19 commits into from
Mar 8, 2016

Conversation

rashidkpc
Copy link
Contributor

This pull significantly tightens up spacing on the dashboard and within visualizations including:

  • Getting rid of padding in the panels and metric vis
  • Removing the 20px high area for the spy toggle and moving it to an absolute positioned element since it is only visible on hover now anyway
  • Move the drag handle for panels off of the large header and to small icon since moving panels is not a task that is done regularly

These changes let you fit a lot more stuff on the dashboard. There's probably more improvements that could be made in this area

screen shot 2016-02-13 at 2 56 49 pm

@panda01
Copy link
Contributor

panda01 commented Mar 2, 2016

This does improve a lot on spacing. Only one nitpick is the move icon doesn't change the cursor on hover. Otherwise, this LGTM!

@panda01 panda01 assigned rashidkpc and unassigned panda01 Mar 2, 2016
rashidkpc pushed a commit that referenced this pull request Mar 8, 2016
Tighten up spacing, move drag handle, move spy toggle
@rashidkpc rashidkpc merged commit 431ba87 into elastic:feature/design Mar 8, 2016
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.

3 participants