mirror of
https://github.com/kusti8/proton-native.git
synced 2026-05-15 14:15:50 -06:00
[PR #133] [MERGED] Add min and max for slider #272
Labels
No labels
bug
documentation
enhancement
libui issue
pull-request
question
wait for libui implementation
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: github-starred/proton-native#272
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/kusti8/proton-native/pull/133
Author: @kusti8
Created: 5/16/2018
Status: ✅ Merged
Merged: 5/21/2018
Merged by: @kusti8
Base:
master← Head:slider-minmax📝 Commits (6)
b613b2aAdd min max support for sliderc6af16cReset demo16ed2d7Don't destroy children elements for slider983943fPrevent slider from resetting when recreatedc9598f4Reset value for slider when beyond max or min. Cleanup code0766636Add to documentation📊 Changes
3 files changed (+98 additions, -19 deletions)
View changed files
📝
docs/component_APIs/slider.md(+35 -17)📝
src/components/DesktopComponent.js(+31 -1)📝
src/components/Slider.js(+32 -1)📄 Description
Fixes #131. Had to do some weird stuff, mainly removing all the children of the Slider's parent in libui, remaking the slider, and then adding back all of those children, which I want to test more thoroughly before I merge.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.