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

[core] feat(NumericInput): disable buttons when value at min/max #4069

Merged
merged 3 commits into from
Apr 20, 2020
Merged

[core] feat(NumericInput): disable buttons when value at min/max #4069

merged 3 commits into from
Apr 20, 2020

Conversation

spencersutton
Copy link
Contributor

Fixes #804

Checklist

  • Includes tests
  • Update documentation

Changes proposed in this pull request:

When the value of a NumericInput is at max disable the increment button and when the value is at min disable the decrement button.

Reviewers should focus on:

Screenshot

Screen Shot 2020-04-17 at 9 29 13 PM

Screen Shot 2020-04-17 at 9 29 57 PM

@spencersutton spencersutton changed the title Disable increment and decrement buttons on NumericInput when at max/min. Disable increment/decrement buttons on NumericInput when at max/min. Apr 18, 2020
Copy link
Contributor

@adidahiya adidahiya left a comment

Choose a reason for hiding this comment

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

nice, thanks @spencersutton

@adidahiya adidahiya changed the title Disable increment/decrement buttons on NumericInput when at max/min. [core] feat(NumericInput): disable buttons when value at min/max Apr 20, 2020
@adidahiya adidahiya merged commit 3a3aadf into palantir:develop Apr 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Numerical input: disable increment / decrement buttons when value at max / min
2 participants