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.

Sign up for GitHub

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

Jump to bottom

Vis spy full screen toggle fixes #1599

Merged
merged 5 commits into from
Oct 10, 2014
Merged

Vis spy full screen toggle fixes #1599

merged 5 commits into from
Oct 10, 2014

Conversation

w33ble
Copy link
Contributor

@w33ble w33ble commented Oct 8, 2014

Closes #1592

  • Don't show full page button when defaulting to full page
  • Expand the number of rows in the table when spy is in fill mode

screenshot 2014-10-08 14 34 32
screenshot 2014-10-08 14 34 42

@w33ble w33ble added the review label Oct 8, 2014
@w33ble w33ble mentioned this pull request Oct 9, 2014
Closed
@rashidkpc
Copy link
Contributor

This LGTM

w33ble added a commit that referenced this pull request Oct 10, 2014
@w33ble
Merge pull request #1599 from w33ble/vis-spy-full
068ccda 
Vis spy full screen toggle fixes
@w33ble w33ble merged commit 068ccda into elastic:master Oct 10, 2014
@w33ble w33ble deleted the vis-spy-full branch February 6, 2015 21:04
@enterstudio enterstudio mentioned this pull request Jul 31, 2022
Open
This was referenced Jul 31, 2022
Open
Open
Open
@larrycameron80 larrycameron80 mentioned this pull request Aug 1, 2022
Open
@snyk-bot snyk-bot mentioned this pull request Aug 1, 2022
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Visualization spy full screen button
2 participants
@w33ble @rashidkpc