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 l'affichage des longs pseudos sur le profil sur mobile #6394

Merged
merged 2 commits into from
Oct 8, 2022

Conversation

Arnaud-D
Copy link
Contributor

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

Fix #5989.

Ça règle les soucis courants. Je n'ai pas géré les cas encore plus extrêmes qui cassent aussi le design sur ordinateur, ça me paraît trop rare pour être traité avant que le cas ne se présente effectivement.

Contrôle qualité

  • Créer un compte avec long pseudo ; j'ai choisi "TrèèèèèèèèèèèèèèsLoooongPseudo".

  • Mettez-lui une sanction et/ou un signalement (important pour voir le comportement des blocs associés).

  • Regarder que la page est affichée normalement sur une largeur ordinateur.

  • Utilisez vos outils développeurs pour réduire la taille jusqu'à des tailles ridicules et constater que le pseudo ne déborde pas (d'autres choses peuvent déborder notamment pour le staff, c'est un autre sujet).

  • Vérifier également que les autres flexpages (la bibliothèque en particulier) fonctionnent bien aussi.

@Arnaud-D Arnaud-D added the C-Front Concerne l'interface du site label Oct 4, 2022
@coveralls
Copy link

coveralls commented Oct 4, 2022

Coverage Status

Coverage remained the same at 88.303% when pulling 4357eec on Arnaud-D:ellipse_pseudos_trop_longs into cb9e83a 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 NOK

Tes modifications sont suffisamment efficaces sur mobile et tablette mais pas assez sur ordinateur car les boutons sur la droite sont tous écrasés !

Capture d'écran avant ma proposition

Ajouter overflow: hidden à l'élément <div class="title"> permet de corriger le soucis.

Capture d'écran après ma proposition

Édition : Je viens de réaliser que le ticket ne concernant que la partie mobile donc on pourrait fusionner tel quel, mais autant corriger aussi la partie ordinateur étant donné que j'ai une solution qui fonctionne !

@Arnaud-D
Copy link
Contributor Author

Arnaud-D commented Oct 7, 2022

Voilà, j'ai fait la modif.

@Arnaud-D Arnaud-D added the hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest label Oct 7, 2022
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 merged commit 08d06b1 into zestedesavoir:dev Oct 8, 2022
@Arnaud-D Arnaud-D deleted the ellipse_pseudos_trop_longs branch October 9, 2022 08:20
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.

Les pseudos trop long débordent du profil
3 participants