Skip to content

[ML] File data visualizer - fixing padding around whole page - #31246

Merged
jgowdyelastic merged 1 commit into
elastic:masterfrom
jgowdyelastic:fixing-padding-in-file-data-vis
Feb 15, 2019
Merged

jgowdyelastic merged 1 commit into
elastic:masterfrom
jgowdyelastic:fixing-padding-in-file-data-vis

Conversation

@jgowdyelastic

Copy link
Copy Markdown
Member

Replaces the padding around the edge of the File Data Visualizer page, removed in #30536.

Before:
image

After:
image

Checklist

Use strikethroughs to remove checklist items you don't feel are applicable to this PR.

For maintainers

- [ ] This was checked for breaking API changes and was labeled appropriately
- [ ] This includes a feature addition or change that requires a release note and was labeled appropriately

@jgowdyelastic jgowdyelastic added review non-issue Indicates to automation that a pull request should not appear in the release notes v7.0.0 :ml cosmetic v8.0.0 v7.2.0 labels Feb 15, 2019
@jgowdyelastic jgowdyelastic self-assigned this Feb 15, 2019
@elasticmachine

Copy link
Copy Markdown
Contributor

Pinging @elastic/ml-ui

@peteharverson peteharverson left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@walterra walterra left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@elasticmachine

Copy link
Copy Markdown
Contributor

💔 Build Failed

@jgowdyelastic

Copy link
Copy Markdown
Member Author

retest

@elasticmachine

Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@alvarezmelissa87 alvarezmelissa87 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM ⚡️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cosmetic :ml non-issue Indicates to automation that a pull request should not appear in the release notes review v7.0.0 v7.2.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants