Press outside of ActionSheet crashs the application
I maintainer di solito rispondono entro 1 giorno
@nitzanyiz ci sta già lavorando.
Dal 17/10/2023.
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
Description
Used the ActionSheet component, when the component is visible and the user press anywhere outside the component the application crashes.
Somehow when this happens the option index sent to the onOptionPress function is always the same as the size of the options array.
options.length = 3, the value sent is 3.
Related to
- Components
- Demo
- Docs
- Typings
Steps to reproduce
- Use the ActionSheet component
- Show the ActionSheet
- Press outside of the ActionSheet component
- See the error
Expected behavior
Hide the ActionSheet.
Actual behavior
Crash the application.
More Info
Code snippet
Current code:
onOptionPress(optionIndex: number) {
this.props.options?.[optionIndex].onPress?.();
this.props.onDismiss?.();
}
Fix:
onOptionPress(optionIndex: number) {
this.props.options?.[optionIndex]?.onPress?.();
this.props.onDismiss?.();
}
Screenshots/Video
Error:
Environment
- React Native version: 0.68.6
- React Native UI Lib version: 6.31.0
Affected platforms
- Android
- iOS
- Web
- Lingua principale
- TypeScript
- Stelle
- 7.2k
- Fork
- 749
- Merge medio
- 29m
- PR unite (30g)
- 1
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di wix/react-native-ui-lib
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
wix/react-native-ui-lib#4022 ·
I maintainer di solito rispondono entro 1 giorno
-
SortableGridList does not forward props to its inner ScrollView (inconsistent with SortableList)Forse già presa @yinminqian l’ha presa 173 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
wix/react-native-ui-lib#4002 ·
I maintainer di solito rispondono entro 1 giorno
-
RN 0.85.0 StyleSheet.absoluteFillObject is removedForse già presa @ChrisJamesC l’ha presa 80 giorni fa. Apertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
wix/react-native-ui-lib#3965 · 3 commenti · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
asBaseComponent: stale module-level colorScheme prevents re-render on scheme switch backForse già presa @huytdps13400 l’ha presa 34 giorni fa. Apertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
wix/react-native-ui-lib#3945 ·
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
wix/react-native-ui-lib#4044 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di wix/react-native-ui-lib
Issue simili
-
Difficoltà 1/5 1-3 ore Idoneità per principianti 86/100
ranxianglei/billion-context#5893 ·
I maintainer di solito rispondono entro 1 giorno
-
stringify escapes leading digits in pseudo-class arguments (:nth-child(1) becomes :nth-child(\31 ))Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 83/100
I maintainer di solito rispondono entro 1 giorno
-
enhancement security severity:low
Difficoltà 2/5 1-3 ore Idoneità per principianti 83/100
Battle-Cattle/BCUK-Bot-4#959 ·
I maintainer di solito rispondono entro 1 giorno
-
[Bug]: Claude account rows no longer show the email (since #8055), so "Account 1, 2, 3" cannot be told apartForse già presa Una pull request collegata a questa issue è aperta o già unita. Apertaaccount-pool bug gui
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
lidge-jun/opencodex#8286 · 2 commenti · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno