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

[BO - Signalement] Affichage des photos dans un album #2158

Merged
merged 7 commits into from
Feb 8, 2024

Conversation

emilschn
Copy link
Collaborator

Ticket

#2095

Description

Mise en place d'un album photo pour visionner les photos liées à un signalement.
Voir "Mode album" dans les specs : https://github.com/MTES-MCT/histologe/wiki/Ajout-documents

  • Quand on clique sur une photo, ça ouvre l'album sur la photo cliquée.
  • Quand on clique sur précédent / suivant, on se déplace dans la liste des photos
  • Des informations sur la photo sont affichées

Tests

  • Ouvrir un signalement fait avec l'ancien formulaire, vérifier que tout s'affiche bien
  • Ouvrir un signalement fait avec le nouveau formulaire, vérifier que tout s'affiche bien
  • Ajouter des photos dans le BO et vérifier que tout s'affiche toujours bien

public/js/app.js Outdated Show resolved Hide resolved
public/js/app.js Outdated Show resolved Hide resolved
Copy link
Collaborator

@numew numew left a comment

Choose a reason for hiding this comment

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

J'ai 2 questions :
On ne fait pas la même chose sur les photo par désordre ?
Screenshot 2024-02-01 at 15-57-31 Histologe

J'ai l'impression que l'affichage est pas le bon (lien bleu foncé sur fond foncé par exemple), mais c'est peut être un soucis chez moi ?
Screenshot 2024-02-01 at 15-54-51 Histologe

@emilschn emilschn force-pushed the feature/2095-signalement-album branch from dcc1f54 to 5f5fc12 Compare February 8, 2024 10:23
@emilschn
Copy link
Collaborator Author

emilschn commented Feb 8, 2024

@numew j'ai changé l'apparence du bouton de fermeture
Et effectivement, les images de désordres ne s'ouvraient pas dans l'album. C'est corrigé :)
Merci !

Copy link

sonarcloud bot commented Feb 8, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link
Collaborator

@numew numew left a comment

Choose a reason for hiding this comment

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

Tests et relecture OK

Copy link
Collaborator

@sfinx13 sfinx13 left a comment

Choose a reason for hiding this comment

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

Lecture + Test OK

@sfinx13 sfinx13 merged commit 4b33a3d into develop Feb 8, 2024
3 checks passed
@sfinx13 sfinx13 deleted the feature/2095-signalement-album branch February 13, 2024 21:09
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.

None yet

3 participants