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

Viewpoints HD to Trek and OutdoorSite detail page #967

Merged
merged 37 commits into from
Feb 7, 2024
Merged

Conversation

dtrucs
Copy link
Collaborator

@dtrucs dtrucs commented Sep 7, 2023

Add viewpoints HD as described in #943

@dtrucs dtrucs changed the base branch from main to develop September 7, 2023 12:58
@babastienne
Copy link
Member

babastienne commented Sep 8, 2023

Quelques retours en vrac sur mes observations en testant à chaud la fonctionnalité :

  • Il manque les vue HD sur les POI
  • Il faudrait pouvoir visualiser les vues HD sur la carte (localisation + pop-up pour ouvrir la photo). (je sais, c'est en attente de GTA pour pouvoir le développer, je ne fais que lister ce qui manque)
  • Quand on pourra localiser les vue HD (point précédent) il faudra les ajouter aux couches activables sur la carte (menu en haut à droite de la carte).
  • L'affichage des boutons zoom / dezoom / plein écran est parfois aléatoire quand on navigue (point remonté par @dtrucs )
  • En mode mobile je trouve la navigation particulière : Je clique sur mon point de vue puis je clique sur la flèche retour qui me ramène au détail de l'objet. Et bien à partir de là j'ai un bouton "Voir la carte" qui, si je clique dessus, ne me montre pas la carte mais la phot HD que j'avais préalablement ouverte. Je trouve que c'est pas hyper intuitif.
  • On avait parlé aussi de pouvoir identifier en haut de la fiche de l'objet via un picto le fait qu'il ai des éléments HD rattaché (pictogramme avec ancre qui pointe vers le contenu) : voir capture ci-dessous (note : il faut qu'on trouve un picto pour illustrer les vues HD)
    image
  • J'ai testé en mode PWA et ça ne fonctionne pas. Il faut pouvoir gérer le cas d'un utilisateur qui souhaite embarquer en mode hors connexion un contenu avec des vues HD. Au même titre qu'on stocke localement les images d'illustration de l'objet on devrait stocker les tuiles de l'Image HD ainsi que ses annotation il me semble, ça me parait important. Si toutefois je suis le seul de cet avis et qu'on estime qu'on ne doit pas embarquer cet élément hors connexion il faut tout de même ne pas l'afficher lorsque l'utilisateur est en mode hors connexion.

@dtrucs
Copy link
Collaborator Author

dtrucs commented Sep 8, 2023

En mode mobile je trouve la navigation particulière : Je clique sur mon point de vue puis je clique sur la flèche retour qui me ramène au détail de l'objet. Et bien à partir de là j'ai un bouton "Voir la carte" qui, si je clique dessus, ne me montre pas la carte mais la phot HD que j'avais préalablement ouverte. Je trouve que c'est pas hyper intuitif.

Tu me feras une démo car en essayant de reproduire ça affiche bien la carte de mon côté

@dtrucs dtrucs merged commit 57795cb into develop Feb 7, 2024
2 checks passed
@dtrucs dtrucs deleted the ViewPointHD branch February 7, 2024 16:55
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.

3 participants