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

feat(depute): gestion des photos #4

Open
revolunet opened this issue Nov 5, 2022 · 12 comments
Open

feat(depute): gestion des photos #4

revolunet opened this issue Nov 5, 2022 · 12 comments
Labels
question Further information is requested UX

Comments

@revolunet
Copy link

A propos des photos des députés :

Il y a sur la version 1 pas mal de traitements "runtime" des images :

  • une notion de "autoflip" qui est un flag dans la table "parlementaires"... je n'ai pas vraiment compris a quoi sert ce flag. peut-être @teymour s'en souvient ?
  • une pastille de couleur ajoutée selon le groupe
  • possibilité de resize/flip via des paramètres d'url

Une possibilité serait, au niveau du repo "data", qu'on s'occupe de les préprocesser en amont pour pouvoir les exploiter de manière statique.

@revolunet revolunet added the question Further information is requested label Nov 5, 2022
@implicitdef
Copy link
Contributor

implicitdef commented Nov 5, 2022

Autoflip c'était pour que les photos "regardent" du même côté. Par défaut sur certaines le député était légèrement vers la droite ou vers la gauche, les anciens voulaient qu'il n'y ait aucun "biais", et donc marquer les photos qu'il fallait flipper (je pense qu'on peut faire sans)

La pastille de couleur ça on peut tout refaire en CSS aucun souci

Je sais pas trop quel est l'approche à suivre. Le plus simple pour démarrer serait de mettre les photos directement dans le dossiers public de Next.js, comme des assets normaux. Le souci c'est qu'on sera pas à jour quand un nouveau député arrive.

Peut-être on pourrait faire une moulinette qui tous les jours va récupérer les dernières images depuis le site de l'assemblée (ou leur open data si c'est dedans ?) et les poser sur un truc type S3 (peut-être avec plusieurs dimensions si besoin) d'où le front pourra les servir directement

@revolunet
Copy link
Author

Ok pour faire simple pour démarrer.

On pourrait faire une commande yarn lancée par une GH action qui va récupérer+traiter (resize) régulièrement les photos depuis le site de l'AN et les commit sur le repo dans /public en cas de changement. je peux proposer une PR ca si ca te parait cohérent.

le truc de la pastille de couleur en CSS oui... il faut "juste" gérer les éventuels différentes tailles, responsive... ca peut faire un composant react sympa :)

@revolunet
Copy link
Author

(Le flip peut aussi être géré en CSS en fait)

@implicitdef
Copy link
Contributor

Yes pour la GH action ça me parait un bon début, tu peux y aller

@revolunet revolunet changed the title feat: gestion des photos feat(depute): gestion des photos Nov 6, 2022
@revolunet
Copy link
Author

je pense qu'on peut close, il ne reste a priori plus que la gestion de la pastille du groupe

@RouxRC
Copy link
Member

RouxRC commented Nov 6, 2022

Hello, je trouvais assez cool personnellement que les gens qui réutilisent l'API de ND présentent du coup sur leurs sites également des photos avec nos pastilles maison (ce qui permettait aussi très rapidement de les identifier), du coup je plaiderai personnellement en faveur de rajouter les pastilles en dur dans votre GH action plutôt qu'en css sur le site

@RouxRC
Copy link
Member

RouxRC commented Nov 6, 2022

(le flip j'aurai tendance à l'oublier par contre ;) )

@revolunet
Copy link
Author

Je pense qu'on peut l'ajouter sur le JPG directement, d'autant que l'info de la "couleur" est fournie directement sur le site de l'AN : https://www2.assemblee-nationale.fr/deputes/liste/photo

@RouxRC est-ce que NosDeputes utilise les memes couleurs que celles de l'AN ?

Capture d’écran 2022-11-06 à 16 09 39

Je peux tenter d'ajouter ca au moment du fetch si ok pour @implicitdef

@RouxRC
Copy link
Member

RouxRC commented Nov 6, 2022

ha non je découvre ces couleurs sur le site de l'AN je ne les avais jamais vues encore.
Non jusque là les couleurs étaient l'un de nos rares choix éditoriaux, réglé en configuration ici : https://github.com/regardscitoyens/nosdeputes.fr/blob/master/config/app.yml.example#L14-L32
(et je les trouve plus justes, j'ai un peu de mal avec le violet pour renaissance côté an je dois avouer...)

@implicitdef
Copy link
Contributor

En fait je ne suis pas très convaincu non plus par ces pastilles en termes d'UX. Je trouve difficile de mémoriser les couleurs, et je n'étais même pas sûr que ça représentait le groupe.
Je pense que c'est quelque chose qu'il faut voir au moment du design de la page, suivant le contexte, est-ce qu'on veut afficher juste une couleur, ou rajouter l'acronyme du groupe, est-ce qu'on veut que ce soit sur l'image, ou à côté, et dans quelles dimensions ? car cela mérite peut-être de changer si l'image fait telle ou telle taille

Bref tu peux le faire si tu veux mais suivant ce qu'on fait niveau design plus tard je ne suis pas sûr que ça restera

@RouxRC
Copy link
Member

RouxRC commented Nov 6, 2022

C'est à vous d'en débattre entre vous clairement. L'argument principal à mon avis de la présence sur les photos, c'est de pouvoir identifier le ou les groupes d'appartenance des députés lorsqu'ils apparaissent dans d'autres contextes, par exemple dans un compte-rendu de débat, dans une liste de signataires d'amendement ou de propositions de lois, etc.

@revolunet
Copy link
Author

ok on met ce sujet à débattre dans la catégorie ux/design :)

@revolunet revolunet added the UX label Nov 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested UX
Projects
None yet
Development

No branches or pull requests

3 participants