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

feat: add slider limits to web #565

Merged
merged 8 commits into from
Jan 4, 2024

Conversation

JacobDel
Copy link
Contributor

@JacobDel JacobDel commented Dec 21, 2023

Continues on abandoned PR: #520

Summary:

Add upperLimit support and lowerLimit support to web.

Test Plan:

Also added an example to the web-example project.
https://github.com/callstack/react-native-slider/assets/38087818/32c09f40-cbc4-4dec-ba51-0cda036ef442

dcangulo and others added 3 commits July 8, 2023 14:25
add a lower/upper limit example to web example
- lower/upp limit variable is optional
- fix computation of newValue
- add if-else case for lower/upper limits
- remove layout changes from commit 641995f
@JacobDel JacobDel changed the title Feat/limits feat: add slider limits to web Dec 21, 2023
Copy link
Member

@BartoszKlonowski BartoszKlonowski left a comment

Choose a reason for hiding this comment

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

This looks great: simple and working. Thank you!
I only have one minor nit regarding naming, as I found it confusing at first and would like to make it a bit more readable - once you accept or anyhow adjust, it's totally ready to be merged.
Thank you 👍

package/src/RNCSliderNativeComponent.web.tsx Outdated Show resolved Hide resolved
package/src/Slider.tsx Outdated Show resolved Hide resolved
MR: refactor valueAfterUpperLimit to valueInLimitRange

Co-authored-by: Bartosz Klonowski <70535775+BartoszKlonowski@users.noreply.github.com>
Copy link
Member

@BartoszKlonowski BartoszKlonowski left a comment

Choose a reason for hiding this comment

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

Some linting corrections.

package/src/Slider.tsx Outdated Show resolved Hide resolved
package/src/Slider.tsx Outdated Show resolved Hide resolved
package/src/Slider.tsx Outdated Show resolved Hide resolved
JacobDel and others added 3 commits December 22, 2023 21:01
linting corrections

Co-authored-by: Bartosz Klonowski <70535775+BartoszKlonowski@users.noreply.github.com>
linting corrections

Co-authored-by: Bartosz Klonowski <70535775+BartoszKlonowski@users.noreply.github.com>
linting corrections

Co-authored-by: Bartosz Klonowski <70535775+BartoszKlonowski@users.noreply.github.com>
@JacobDel
Copy link
Contributor Author

I'm not sure how to fix the Lint issue

@BartoszKlonowski
Copy link
Member

@JacobDel Normally you should run npx eslint --fix 'file' for the 'file' you would like to lint.
I decided to do that through GH PR interface, hoping it will be faster, but I made the simple mistake of missing indentation.
I already fixed it.

@BartoszKlonowski BartoszKlonowski merged commit 8bbdff1 into callstack:main Jan 4, 2024
8 checks passed
@JacobDel
Copy link
Contributor Author

Do you have an estimate when this would be released?

@BartoszKlonowski
Copy link
Member

Hello @JacobDel! Most probably next Friday (23.02).

@JacobDel
Copy link
Contributor Author

Hi, any updates on this? Is there a way to build and use it locally?

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.

None yet

3 participants