Comments (13)
You can add loading prop. It's not documented currently but should be
from react-native-elements.
@Jeanmsilva89 I really like what you have done with the activity indicators on the button elements. gj!
What other components were you thinking of adding the activity indicator too?
Also I think, as part of your PR you forgot to update the readme with the new button props that you implemented. Feel free to submit a PR for this if you are not busy.
from react-native-elements.
@Monte9 Sorry I have been too busy in the past few weeks, and missed your comment. I'll send a PR for the README by this weekend.
@dekm I'm not using the SearchBar currently but yeah I think it's useful for just about every component that has an onPress event so that a loading state can be added.
from react-native-elements.
@Monte9 SearchBar has a activity indicator, was recently added.
I agree with changing socialIcon to Button
from react-native-elements.
I think this would be a great addition for sure! @Jeanmsilva89
from react-native-elements.
I'd love something like this for my project!
from react-native-elements.
Added pull request #62
from react-native-elements.
I'd love to add this feature to other components, and I'll have to do it for my own projects so I'll submit another PR soon for that.
@dabit3 Another topic but figured I'd ask you here instead of opening another issue.
Is there a reason for why you decided to create the SocialIcon component from scratch as opposed to building it on top of your button component? In a lot of ways the social media icons are used as buttons and could share lot of the button features.
For example. I just created a branch where I replicated the props. "small, disabled, loading" into the socialIcon component. I initially assumed it was built using the button so I thought it would've just worked.
Btw thanks a lot for this cool project and for you awesome React Native Radio, been really enjoying it big time. Keep up!
from react-native-elements.
Would be cool to have this added to the Searchbar as well. I was just trying to look into adding a spinner on the right side of the searchbar as I'm calling search as the user types.
Looks like inside the search component there is only the left icon currently added.
from react-native-elements.
@Jeanmsilva89 any updates?
from react-native-elements.
@binoy14 since we haven't heard from Jean, maybe this is something we can do ourselves. We can talk about what elements it would make sense to add the activityIndicator to.
Also, reading his comment above about SocialIcon not built on top of Button.. it makes sense. I wonder why this is case? Could we possibly refactor?
from react-native-elements.
Closing this issue as PR (#62) has been merged
The Button -> SocialIcon refactor is unrelated to this issue. @binoy14 I'll follow-up with you on this and put it on our TODO list.
from react-native-elements.
How can I add a spinner to a button conditionally ?
from react-native-elements.
Related Issues (20)
- import { styled } from '@rneui/themed' doesn't work HOT 2
- Feature Request - Animated Shrink Button with Loading Indicator
- Avatar ignoring resize mode on initial load HOT 18
- ButtonGroup fails on iOS Expo Dev Client HOT 1
- Feature request - Circular progress component
- The documentation sidebar is not taking the correct height on mobile screens.
- The search glass is not available in Chrome browser
- Dialog.Title does not honor Dark mode text color HOT 3
- SpeedDial cant work in web platform,cuz ``` [...new Array(React.Children.count(children))]``` will returned ```[empty × 2]``` HOT 2
- NPM version missmatch for @rneui/themed's peerDeps/devDeps HOT 1
- Unable to import styled helper HOT 1
- ref prop is not working for Button component of New React native element lib (@rneui/base & @rneui/themed) HOT 2
- Upgrade Expo version for Expo demo HOT 1
- [Feature Request] For web renders, let `Icon` and `Chip` render as `<button>` only if it needs to be clickable HOT 1
- Latest @rneui/themed has incorrect peer and dev dependency HOT 1
- SearchBar has no Cancel button on iOS HOT 3
- TextProps are extended by ListItem.Content instead of ListItem.Title
- Show default image when source load fail HOT 1
- Cannot use Button as child of Link. HOT 1
- clearIcon in SearchBar component does not theme for dark mode (still appears light)
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from react-native-elements.