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

Dashboard legend height problems #6935

Merged
merged 1 commit into from
Apr 27, 2016
Merged

Conversation

panda01
Copy link
Contributor

@panda01 panda01 commented Apr 14, 2016

Fix the legend on dashboard so that way it scrolls when it's too big and the vis takes up a reasonable position.

closes #6772.

…g stubbonr, suspect something to do with the visualize element not being native html
@panda01 panda01 self-assigned this Apr 14, 2016
@panda01 panda01 added the v5.0.0 label Apr 14, 2016
@panda01 panda01 changed the title [WIP] Fixing problems with vis height Dashboard legend height problems Apr 25, 2016
@panda01 panda01 assigned w33ble and unassigned panda01 Apr 26, 2016
@w33ble w33ble added the review label Apr 26, 2016
@w33ble
Copy link
Contributor

w33ble commented Apr 27, 2016

Ooooh, you fixed an issue with the app header in visualize too, eh? Nice!

Before
apr-27-2016 11-26-30

After
apr-27-2016 11-28-46

@w33ble
Copy link
Contributor

w33ble commented Apr 27, 2016

LGTM!

@w33ble w33ble merged commit 1d9a008 into elastic:master Apr 27, 2016
epixa added a commit to epixa/kibana that referenced this pull request Apr 29, 2016
ycombinator added a commit that referenced this pull request Apr 29, 2016
Revert "Merge pull request #6935 from panda01/fix/superTallVis"
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.

Pie chart positioning on Dashboard seems wrong
3 participants