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

Refactor SliderInput #236

Merged
merged 1 commit into from
Jan 2, 2023
Merged

Refactor SliderInput #236

merged 1 commit into from
Jan 2, 2023

Conversation

willeastcott
Copy link
Contributor

  • Eliminate uses of Object.defineProperty to programmatically add getters/setters to SliderInput. This prevents them from appearing in the docs or the types.
  • Remove the pre property of the SliderInput args object (there for backwards compatibility of precision). I don't see it used by any other projects.
  • Remove the utility proxy function which is no longer used.

@willeastcott willeastcott added the enhancement New feature or request label Jan 1, 2023
@willeastcott willeastcott requested a review from a team January 1, 2023 23:16
@willeastcott willeastcott self-assigned this Jan 1, 2023
@willeastcott willeastcott merged commit be1427c into main Jan 2, 2023
@willeastcott willeastcott deleted the refactor-sliderinput branch January 2, 2023 23:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants