-
Notifications
You must be signed in to change notification settings - Fork 106
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
fix: Back Button not working in IdentificationModal (#2764) #4467
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #4467 +/- ##
==========================================
- Coverage 47.84% 46.83% -1.01%
==========================================
Files 1530 1676 +146
Lines 31565 29234 -2331
Branches 7933 7364 -569
==========================================
- Hits 15103 13693 -1410
+ Misses 16406 15491 -915
+ Partials 56 50 -6
... and 1267 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
Ne approfitto per chiedere come mai la schermata di identificazione tramite pin è stata implementata come Modal invece di uno Screen di react-navigation? È una cosa che mi sono sempre chiesto |
This pull request is stale because it has been open for 60 days with no activity. If the pull request is still valid, please update it within 14 days to keep it open or merge it, otherwise it will be closed automatically. |
This pull request is stale because it has been open for 60 days with no activity. If the pull request is still valid, please update it within 14 days to keep it open or merge it, otherwise it will be closed automatically. |
This pull request is stale because it has been open for 60 days with no activity. If the pull request is still valid, please update it within 14 days to keep it open or merge it, otherwise it will be closed automatically. |
This pull request was closed because it has been inactive for 14 days since being marked as stale. |
Short description
Chiude #2764. Permette la chiusura dell'app su Android premendo il tasto indietro quando si è nella Schermata di Inserimento del PIN proprio come avviene nelle altre schermate.
How to test
Ho testato il comportamento del pulsante indietro nelle altre schermate ed è il medesimo.
Notes
Mi piacerebbe modificare il comportamento del pulsante "indietro" nelle altre schermate per fare in modo che venga richiesto il doppio tap per uscire (come proposto qui da @drsurfer). Non trovo, però, il codice dove viene gestita l'uscita nelle altre schermate (forse è un comportamento di default di React Native - che non ho mai utilizzato prima - o forse è la mia pigrizia domenicale che non ha aiutato). Mi piacerebbero le vostre opinioni rispetto a questa potenziale modifica e, eventualmente, degli indizi su cosa e dove modificare.