search

tchapgouv / tchap-landing-page

Tchap landing page
0 stars 6 forks source link

Add faq page for password reset #161

Closed aulamber closed 1 year ago

aulamber commented 1 year ago

Add an additional page to the current /faq, called /faq-mot-de-passe. You can navigate there from :

Screenshot 2023-02-06 at 09 05 40

And then the content of the faq reset password is :

Screenshot 2023-02-06 at 16 35 32
areox-net commented 1 year ago

I think the idea of having a dedicated page is good. However, I don't think we should use this cascade format. The full reset sequence should be visible in its entirety for each platform in one single cascade module. Actually, the page should not have cascades at all. It should just be 3 tabs, one for each platform.

aulamber commented 1 year ago

I think the idea of having a dedicated page is good. However, I don't think we should use this cascade format. The full reset sequence should be visible in its entirety for each platform in one single cascade module. Actually, the page should not have cascades at all. It should just be 3 tabs, one for each platform.

Thank you for your feedback :) I applied the organization I found on the new PDF, but I will change that to tabs this morning.

odelcroi commented 1 year ago

looks good to me 👍

aulamber commented 1 year ago

looks good to me 👍

  • en amélioration "nice to have" (à lister dans un ticket et on le fera quand on a le temps)

    • image responsive?
    • ajouter une ancre pour avoir des liens facilement partageable pour un appareil donné

Pour l'image responsive, c'est une bonne idée mais pour cela il me faudrait des svg plutôt que des png. Pour l'ancre, je l'ai fait pour chacun des trois tabs (web, android, ios), mais pas pour chaque point de chaque tab, cela ne me paraissait pas nécessaire étant donné ce sont plusieurs étapes pour un seul résultat, et pas des sujets différents. Qu'en penses-tu @odelcroi ?

Aussi, si tu peux valider cette PR c'est top ;) Ou @areox-net ? Merci d'avance !

aulamber commented 1 year ago

I added some improvemebts to the title and first sentence for more clarity.

Concenrning the images, we don't need SVG but just high-resolution PNGs. We can fix this in a new PR. Let's push this for now.

Super, merci ! Je merge :)