Skip to content

fix: Add SheetModal and refactor modals & ED 2FA UI#736

Open
Fefedu973 wants to merge 1 commit intoPapillonApp:devfrom
Fefedu973:fix-android-modal-close-flash
Open

fix: Add SheetModal and refactor modals & ED 2FA UI#736
Fefedu973 wants to merge 1 commit intoPapillonApp:devfrom
Fefedu973:fix-android-modal-close-flash

Conversation

@Fefedu973
Copy link
Copy Markdown
Contributor

Introduce a new SheetModal component (Android-friendly animated sheet) and replace raw Modal usages with SheetModal in several screens (ED credentials, edit_subject, transport). Refactor the ÉcoleDirecte login flow: extract EDDoubleAuthModal, replace animated list item implementation with a sheet-based selectable list, add selection/submission state handling, and queue 2FA responses into the existing login flow. Update edit_subject header height calculation and minor layout tweaks. Add localized strings for double-auth description (en/fr) and import/update various UI primitives (List, Button, Typography, Papicons) and helper utilities.

Contribution

Règles de contribution

Caution

Afin de garantir une application stable et pérenne dans le temps, nous t'invitons à vérifier que tu as bien respecté les règles de contribution. Sans cela, ta Pull Request ne pourra pas être examinée.

  • Cette Pull Request porte sur une seule fonctionnalité ou un seul correctif.
  • Cette Pull Request n'est pas faite essentiellement avec de l'IA.
  • Pour tout changement majeur, j’ai créé une issue afin d’échanger avec les mainteneurs de Papillon sur la meilleure façon de l’intégrer.
  • Ma Pull Request respecte les conventions Conventional Commits et Conventional Branch ainsi que les conventions de codage de l'application.
  • J’ai testé mes modifications sur iOS et Android, et l’application fonctionne correctement.
  • J’emploie un langage informel, clair et concis dans mes messages.
  • J’ai documenté mes changements de manière appropriée, soit dans la description de la Pull Request, soit dans le GitBook.
  • J’ai ajouté les traductions nécessaires dans au moins un fichier de langue.

Introduce a new SheetModal component (Android-friendly animated sheet) and replace raw Modal usages with SheetModal in several screens (ED credentials, edit_subject, transport). Refactor the ÉcoleDirecte login flow: extract EDDoubleAuthModal, replace animated list item implementation with a sheet-based selectable list, add selection/submission state handling, and queue 2FA responses into the existing login flow. Update edit_subject header height calculation and minor layout tweaks. Add localized strings for double-auth description (en/fr) and import/update various UI primitives (List, Button, Typography, Papicons) and helper utilities.
@Fefedu973
Copy link
Copy Markdown
Contributor Author

fixes #719

Copy link
Copy Markdown
Contributor

@ecnivtwelve ecnivtwelve left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sympa, bonne initiative, mais dans l'idée d'utiliser un maximum de code natif avec l'arrivée d'Expo 55, ça pourrait être une bonne alternative d'explorer les Bottom Sheets natives :

De manière à ce qu'on ne multiplie pas trop les composants à maintenir sur la durée. Peut être le wrapper dans un <NativeSheet> avec le BottomSheet iOS ou la Modal React Native

@Fefedu973
Copy link
Copy Markdown
Contributor Author

Ok à voir !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants