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

Improve the default panel size #9775

Merged

Conversation

stacey-gammon
Copy link
Contributor

The default panel size is almost always too small for anyone’s needs. Using half the width per vis seems to work better, along with making them slightly taller.

Three visualizations added with the new defaults:
screen shot 2017-01-06 at 2 37 45 pm

Three visualizations added with the old defaults:
screen shot 2017-01-06 at 2 38 13 pm

The default panel size is almost always too small for anyone’s needs.
Using half the width per vis seems to work better, along with making
slightly taller.
@tbragin
Copy link
Contributor

tbragin commented Jan 8, 2017

@stacey-gammon Tried out this PR and I like this change. Incredible how a much more sensible default improves the experience!

Copy link
Contributor

@tbragin tbragin left a comment

Choose a reason for hiding this comment

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

@tbragin tbragin added Feature:Dashboard Dashboard related features :Sharing labels Jan 8, 2017
Copy link
Contributor

@kobelb kobelb left a comment

Choose a reason for hiding this comment

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

LGTM - totally into this.

@stacey-gammon
Copy link
Contributor Author

All Selenium tests are failing due to #9763 so going to go ahead and merge this.

@stacey-gammon stacey-gammon merged commit a8ae8a7 into elastic:master Jan 9, 2017
elastic-jasper added a commit that referenced this pull request Jan 9, 2017
Backports PR #9775

**Commit 1:**
Improve the default panel size

The default panel size is almost always too small for anyone’s needs.
Using half the width per vis seems to work better, along with making
slightly taller.

* Original sha: b6740f1
* Authored by Stacey Gammon <gammon@elastic.co> on 2017-01-06T19:42:39Z
stacey-gammon added a commit to stacey-gammon/kibana that referenced this pull request Jan 9, 2017
The default panel size is almost always too small for anyone’s needs.
Using half the width per vis seems to work better, along with making
slightly taller.
stacey-gammon added a commit that referenced this pull request Jan 9, 2017
* Improve the default panel size (#9775)

The default panel size is almost always too small for anyone’s needs.
Using half the width per vis seems to work better, along with making
slightly taller.

* Fix selenium tests
@stacey-gammon stacey-gammon deleted the improve-default-panel-size branch January 20, 2017 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Dashboard Dashboard related features review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants