Skip to content

Ajout de la page Synthèse#579

Merged
bellangerq merged 41 commits intomainfrom
488-accès-à-laudit-et-aux-livrables
Dec 14, 2023

Hidden character warning

The head ref may contain hidden characters: "488-acc\u00e8s-\u00e0-laudit-et-aux-livrables"
Merged

Ajout de la page Synthèse#579
bellangerq merged 41 commits intomainfrom
488-accès-à-laudit-et-aux-livrables

Conversation

@bellangerq
Copy link
Copy Markdown
Collaborator

@bellangerq bellangerq commented Dec 6, 2023

  • Ajouter le lien de retour à la synthèse sur la page du rapport (on a pas le editUniqueId)
  • Retirer le lien retour du rapport (même connecté)
  • Toujours ouvrir le rapport dans un nouvel onglet (même connecté)

Pour les notes :

  • Wording du toast de succès
  • Hauteur du <textarea>
  • Modal du Markdown (déjà dans une modale !)

closes #488
closes #558

@netlify
Copy link
Copy Markdown

netlify Bot commented Dec 6, 2023

Deploy Preview for confiture ready!

Name Link
🔨 Latest commit 982d00d
🔍 Latest deploy log https://app.netlify.com/sites/confiture/deploys/657b045802568400084ea20e
😎 Deploy Preview https://deploy-preview-579--confiture.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@bellangerq bellangerq force-pushed the 488-accès-à-laudit-et-aux-livrables branch from 8154c64 to e6e9d30 Compare December 6, 2023 13:19
@bellangerq
Copy link
Copy Markdown
Collaborator Author

bellangerq commented Dec 6, 2023

@AdrienMuzyczka, j'ai bien avancé et j'ai quelques retours :

  • Que fait-on tu bouton "Annuler" de l'étape 1 quand on est en mode édition ? J'imagine qu'on le supprime. // On garde et on renvoie vers la synthèse
  • Je ne trouve pas l'icône file-edit-line pour le bouton "Accéder" du bloc Audit. En attendant j'ai mis le file-line. // icone des notes
  • La barre de progression diffère très peu (épaisseur) de celle qui est dans le sticky header. Est-ce qu'on harmonise les 2 ? Si oui, sur quel modèle ? Si non, pas de soucis, on peut faire 2 variantes. // garder les 2 barres
image image
  • Pareil pour les blocs "Copier le lien" présents dans les lbocs "Rapport" et "Déclaration". Ils changent très peu par rapport à ceux présents sur l'actuelle page de synthèse et dans la décla. Est-ce qu'on met à jour les anciens avec le nouveau design ? // on prend le nouveau
image image

Tu peux déjà faire quelques tests si tu veux. Il y a 2-3 trucs qui sont encore à faire (genre la gestion de la date "Terminé/Créé...") et pour l'instant on peut y accéder seulement après l'étape 1 (ou en changeant l'URL et en mettant /synthese).

@bellangerq bellangerq force-pushed the 488-accès-à-laudit-et-aux-livrables branch from 0162679 to ba2baa0 Compare December 8, 2023 11:10
@bellangerq bellangerq marked this pull request as ready for review December 8, 2023 14:26
@bellangerq bellangerq force-pushed the 488-accès-à-laudit-et-aux-livrables branch from 89ca54e to 0a3e210 Compare December 11, 2023 16:47
Comment thread confiture-rest-api/src/audits/audit.service.ts Outdated
Comment thread confiture-rest-api/templates/audit-creation.mjml Outdated
@benoitdequick
Copy link
Copy Markdown
Contributor

benoitdequick commented Dec 12, 2023

Les icônes Capture d’écran 2023-12-12 à 18 38 07 et Capture d’écran 2023-12-12 à 18 38 47 sont masquées aux AT. Elles apportent quand même une information significative : Ajouter une alternative textuelle : Fait / Pas fait...

Sur les liens consulter, ajouter :

  • le <span>Nouvelle fenêtre</span>
  • un attribut title="consulter le rapport - nouvelle fenêtre"

Pour le partage

  • Sur les liens de partage, préciser dans le title de quel lien de partage il s'agit
  • Sur les boutons Copier le lien, préciser le title

@bellangerq bellangerq merged commit efddcef into main Dec 14, 2023
@bellangerq bellangerq deleted the 488-accès-à-laudit-et-aux-livrables branch December 14, 2023 13:34
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.

Déplacement du bouton "Notes" Accès à l'audit et aux livrables

3 participants