Skip to content

[ML] Fixing metric select double caret - #30357

Merged
jgowdyelastic merged 1 commit into
elastic:masterfrom
jgowdyelastic:fixing-metric-select-double-caret
Feb 7, 2019
Merged

jgowdyelastic merged 1 commit into
elastic:masterfrom
jgowdyelastic:fixing-metric-select-double-caret

Conversation

@jgowdyelastic

@jgowdyelastic jgowdyelastic commented Feb 7, 2019 •

Copy link
Copy Markdown
Member

Removing which was adding an additional caret to the metric selects in the multi-metric and population wizards.

image

Rule originally added in this PR: #19816
Which is now redundant due to this PR: #29658

Checklist

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

@elasticmachine

Copy link
Copy Markdown
Contributor

Pinging @elastic/ml-ui

@jgowdyelastic
jgowdyelastic requested a review from a team as a code owner February 7, 2019 09:20

@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 Succeeded

@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. Works on IE11 too.

@ryankeairns ryankeairns 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

@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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants