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

Allow min and max to have same value #114

Merged

Conversation

jhblacklock
Copy link
Contributor

#55 and #92.

lastKeyMoved sets which slider gets rendered last.

@jhblacklock
Copy link
Contributor Author

@davidchin Any chance this can be added to the next release? We'd love to help if we can.

@davidchin
Copy link
Owner

davidchin commented Jan 5, 2018

Sure. And thank you for your work. Can you add a test case verifying that min/max can be the same? Also, can you add a prop so that your change is backward compatible?

@ghost
Copy link

ghost commented Jan 5, 2018

@davidchin We also have a requirement to set max and min value to same value. As I can tell it is planned that this feature is added to the component? (if yes, when?) thank you.

@jhblacklock
Copy link
Contributor Author

@davidchin tests and prop added. Thanks much!

@davidchin davidchin merged commit 8b36de7 into davidchin:master Jan 6, 2018
@davidchin
Copy link
Owner

Thanks @jhblacklock, your changes have been released as v1.3.0

oyeanuj added a commit to oyeanuj/react-input-range that referenced this pull request Mar 8, 2018
… into upgraded

* 'master' of https://github.com/davidchin/react-input-range: (34 commits)
  Release v1.3.0
  chore: Run test and generate changelog before tagging new version
  chore: Add `allowSameValues` to TypeScript definition
  docs: Add `allowSameValues` property to README
  feat: Allow min and max to have same value (davidchin#114)
  Release v1.2.2
  chore: Add version prefix
  style: Fix comment indentation
  Update package.json (davidchin#111)
  Release v1.2.1
  fix: Always set slider dragging back to false (davidchin#98)
  docs: Remove deprecated prop from example
  docs: Add description for new prop
  Release v1.2.0
  fix: Add missing prop in definition file
  feat: Add track dragging functionality (davidchin#91)
  Release v1.1.5
  fix: Typescript definition file
  style: Move setup block into describe block
  Release v1.1.4
  ...
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.

2 participants