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)
- Latest @rneui/themed has incorrect peer and dev dependency HOT 4
- SearchBar has no Cancel button on iOS HOT 4
- 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 2
- clearIcon in SearchBar component does not theme for dark mode (still appears light)
- Virtualized List error when adding a flatlist inside bottomsheet. HOT 1
- [Feature Request]Classification selector
- Slider disabled prop is not working. HOT 2
- CheckBox icon cannot be seen HOT 1
- Warning: Dialog.Loading: Support for defaultProps will be removed from function components in a future major release. HOT 19
- Add `ScrollViewProps` or `ScrollViewStyle` to Tab component props
- DimensionValue typing HOT 1
- Avatar maintains image despite setting to null HOT 3
- BottomSheet : The onBackdropPress action is not launched on the new architecture
- Moving to draft as we're waiting for a screenshot
- @types/react-native-vector-icons has dependency of deprecated @types/react-native
- ButtonGroup: allow style props to be methods
- Issue with Dialog.Button - Support for defaultProps will be removed
- Skeleton width and height typings are number | undefined (missing string)
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.