Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Set min property on interval input #39

Merged
merged 2 commits into from
Apr 23, 2019

Conversation

dbbaughe
Copy link
Contributor

Issue #, if available:
#19

Description of changes:
Sets a min property in the input element for interval so arrow button/keys don't allow you to go below 1.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@dbbaughe dbbaughe requested a review from mihirsoni April 22, 2019 22:25
Copy link
Contributor

@mihirsoni mihirsoni left a comment

Choose a reason for hiding this comment

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

LGTM!

@jinsoor-amzn jinsoor-amzn self-requested a review April 23, 2019 20:14
Copy link

@jinsoor-amzn jinsoor-amzn left a comment

Choose a reason for hiding this comment

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

👍

@dbbaughe dbbaughe merged commit 5d4fb11 into opendistro-for-elasticsearch:master Apr 23, 2019
@dbbaughe dbbaughe deleted the negative-input branch April 23, 2019 20:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants