Skip to content
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

Implémenter l'annuaire des forums dans Tchap X #35

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

NicolasBuquet
Copy link
Collaborator

Fix #5

L'écran d'annuaire des forums existe déjà dams Element X : il est accessible depuis le champ de recherche sur l'écran principal à condition que ce champ, une fois activé soit vide :

image

Cette vue Annuaire est affiché en plein écran.

Il fallait donc l'appeler depuis ce nouvel écran Tchap X :

image

Le problème rencontré est que cet écran est déjà présenté en plein écran, et qu'il n'est pas possible de présenter sous SwiftUI 2 vues successives en plain écran,
De plus, la vue Annuaire ne semble pas être faite pour être dans une NavigationView.

La vue a donc été présentée à l'écran dans une sheet après être revenu sur l'écran principal de l'application.

Cette vue Annuaire est :

  • pilotée par la classe RoomDirectorySearchScreenCoordinator
  • et les actions presentRoomDirectorySearch et dismissedRoomDirectorySearchScreen

Il a été nécessaire d'introduire le paramètre isPresentedInFullScreenMode sur l'action dismissedRoomDirectorySearchScreen pour appeler la bonne méthode de clôture de vue en fonction de si cette vue a été présentée en mode fullscreen ou en mode sheet.

Voici l'écran final d'annuaire Tchap X présentée en modal sheet :

image

@NicolasBuquet NicolasBuquet added enhancement New feature or request pr-change labels Nov 28, 2024
@NicolasBuquet NicolasBuquet linked an issue Nov 28, 2024 that may be closed by this pull request
2 tasks
Copy link

Warnings
⚠️ Some of the commits are missing ticket numbers. Please consider squashing all commits that don't have a tracking number.
⚠️ Please add a sign-off to either the PR description or to the commits themselves.
⚠️ You seem to have made changes to views. Please consider adding screenshots.

Generated by 🚫 Danger Swift against 81607e5

@NicolasBuquet
Copy link
Collaborator Author

@areox-net Si tu passes par là, tu peux jeter un oeil à cette proposition ? ^ ^

@areox-net
Copy link
Collaborator

Merci @NicolasBuquet, c'est top. Il y aura surement un travail à faire sur le wording.

Avec @Delphgateau on aurait tendence à ne plus parler d'annuaire pour les salons, pour éviter toute ambiguïté avec l'annuaire des agents.
On aura donc un bouton Parcourir les salons publics
et la page Liste des salons publics

Pour simplifier je propose que autant les boutons que la page s'appellent Rejoindre un salon public

Que en penses-tu @Delphgateau ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request pr-change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implémenter l'annuaire des forums dans Tchap X
2 participants