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

Met à jour la page d'accueil #6696

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

Conversation

Arnaud-D
Copy link
Contributor

@Arnaud-D Arnaud-D commented Dec 30, 2024

Cette PR mets à jour la page d'accueil.

  • fait correspondre la structure de la page d'accueil à la future organisation "validé/libre" pour les publications
  • rend la page plus joueuse avec des "call to actions"
  • regroupe les paramètres de configuration dans leur propre section
  • refactorise les fonctions de récupération de données pour qu'elles ne dépendent plus directement de certaines configurations

C'est dans le cadre de la refonte du parcours de validation. J'aimerais découper ça en morceaux digestes pour le développement et la QA. Je pense que cette partie es autoporteuse.

image

Contrôle qualité

Vérifier que le bloc "pépites de l'équipe" contient bien à la fois les tutos et les articles.

Vérifier que les liens fonctionnent

Vérifier les différentes tailles d'écran et que ça reste lisible

Vérifier que j'ai pas cassé la page de profil en refactorisant les paramètres de configuration

@Arnaud-D Arnaud-D added the C-Front Concerne l'interface du site label Dec 30, 2024
@coveralls
Copy link

coveralls commented Dec 30, 2024

Coverage Status

coverage: 89.209% (+0.009%) from 89.2%
when pulling 44a4148 on Arnaud-D:maj-page-accueil
into 4d9a460 on zestedesavoir:dev.

* fait correspondre la structure de la page d'accueil à la future organisation "validé/libre" pour les publications
* rend la page plus joueuse avec des "call to actions"
* regroupe les paramètres de configuration dans leur propre section
* refactorise les fonctions de récupération de données pour qu'elles ne dépendent plus directement de certaines configurations
@firm1
Copy link
Contributor

firm1 commented Jan 22, 2025

Rapport de QA

J'ai pu tester pas mal de point même ceux qui ne sont pas mentionnés dans les conseil QA (par exemple publier plusieurs contenu en majeurs/mineures pour vérifier que l'ordre d'affichage est bien correct).

Cohérence du ton

le ton/style des boutons "Montre-les moi !" et "Fais voir !" dénote un peu avec le bouton "Commencer à rédiger". Il faudrait harmoniser le ton des boutons.

La place du bouton "Commencer a rédiger"

Vu la place du bouton "Commencer a rédiger" qui maintenant sous le titre "Dernières publications de la communauté", on pourrait supposer que dans le bloc, lorsqu'on lit "Il y a X publications sur Zeste de Savoir. Pourquoi pas la vôtre ?".

Le X devrait correspondre aux publications de la communauté "non validés par l'équipe". Mais actuellement le X fais référence à l'ensemble des contenus publiés (validés ou non) sur le site. Cela peut prêter à confusion.

Je dis cela, mais je n'ai pas réfléchi à la bonne façon de traiter le point. je vois au moins 2 options possibles :

  • Avoir 2 blocs "commercer à rédiger" (pour les pépites validées et les publication de la communauté) => ça peut alourdir la page
  • Avoir un bloc "commercer à rédiger" mais qui se détache clairement de l'une des 2 catégories (validées et non validées) => besoin de compétence d'UX/design pour que ce soit bien intégré

Certains boutons n'ont pas besoin d'affichage sur mobile.

Aujourd'hui (en prod) le bouton "Commencer à rédiger" n'est pas affiché lorsqu'on est sur la version mobile du site.

Cette PR affiche ce bouton alors qu'il y a très peu de chance qu'il soit utile sur mobile.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Front Concerne l'interface du site
Projects
Status: Modification demandée
Development

Successfully merging this pull request may close these issues.

3 participants