-
-
Notifications
You must be signed in to change notification settings - Fork 82
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
Component viabilities #21
Comments
Hey, thanks for your interest!
|
Thinking about As for control statements, similar to above, I personally find them useful as HOCs for composition, but they don't need to be in this library. Thanks! I'll start on a PR :) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Howdy! Thanks for starting this. Before I did a bunch of work, I wanted to put forward some suggestions for components to see if you'd be interested in adding them to libreact. If there's interest, I'll definitely make some PRs!
TouchSensor
: This would pass through a boolean indicating if the device supports touch interactions or not. Useful for displaying some UI components on a mobile device rather than guessing a good width.OSOnly
: The OS of the device. This would be helpful for changing/adjusting layout based on Safari's bottom bar, etc. Takes an array of OS' to target.ReducedMotionSensor
: Renders via a child function that passes through whether the user prefers reduced animation on their device. Name isn't the best. Probably just exports a specific instance ofMediaSensor
js-control-statements
would be willing to merge with this project!Tabs
: I don't like the name, but this would provide a way to have a nav that toggles content below without (or maybe with?) changing the route.LoadingSpinner
: Some sort of nice default for a loading spinner.The text was updated successfully, but these errors were encountered: