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

Corrige la taille des images de profil par défaut #6398

Merged
merged 4 commits into from
Oct 11, 2022

Conversation

Arnaud-D
Copy link
Contributor

@Arnaud-D Arnaud-D commented Oct 7, 2022

Reprise de #5467.

Les avatars par défaut sont trop petit sur la page de profil, cette PR les remet à une bonne taille.

J'ai opté pour une option à base de templatag, que j'ai généralisée à tous les endroits que j'ai pu trouver où on affiche les avatar. Un argument par défaut permet de rester léger sur la syntaxe dans les templates.

J'ai aussi modifié un test en conséquence en le rendant moins spécifique volontairement. Je ne crois pas qu'on ai besoin de reproduire le code fonctionnel dans le test pour vérifir que l'adresse exacte est bien la bonne.

Contrôle qualité

Se promener sur le profil et voir que les avatars par défaut sont à la bonne taille.

Se promener partout où on rencontre des avatars (pages de tutos pour les auteurs, commentaire, notifications, MP, forums, etc.) et voir que rien n'est cassé.

@Arnaud-D Arnaud-D added the C-Front Concerne l'interface du site label Oct 7, 2022
@Arnaud-D Arnaud-D added the hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest label Oct 8, 2022
@coveralls
Copy link

coveralls commented Oct 9, 2022

Coverage Status

Coverage increased (+0.005%) to 88.308% when pulling 44c753a on Arnaud-D:taille-avatar into 9cab53f on zestedesavoir:dev.

Copy link
Member

@Situphen Situphen left a comment

Choose a reason for hiding this comment

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

QA OK

@Situphen Situphen enabled auto-merge (squash) October 11, 2022 18:01
@Situphen Situphen merged commit 0eb7143 into zestedesavoir:dev Oct 11, 2022
@Arnaud-D Arnaud-D deleted the taille-avatar branch October 11, 2022 18:39
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 hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants