#27 Les tooltips de la barre de gauche sont décalés sur petit écran

Closed
opened 3 months ago by Ghost · 10 comments
Ghost commented 3 months ago

Sur un écran de petite hauteur (ou quand la hauteur attribuée au site est petite), la barre est scrollable pour pouvoir atteindre tous les éléments.

Le souci, c’est que les tooltips associés aux éléments ne suivent pas le scroll comme on peut le voir dans l’image jointe.

Sur un écran de petite hauteur (ou quand la hauteur attribuée au site est petite), la barre est scrollable pour pouvoir atteindre tous les éléments. Le souci, c'est que les tooltips associés aux éléments ne suivent pas le scroll comme on peut le voir dans l'image jointe.
Lephenixnoir commented 3 months ago
Owner

Ooouh, bien vu ça. Ça a pas l’air trivial. @Darks , est-ce que placer les tooltips relativement au <ul> qu’est le menu ne permettrait pas de les faire scroller au passage ?

Ooouh, bien vu ça. Ça a pas l'air trivial. @Darks , est-ce que placer les tooltips relativement au `<ul>` qu'est le menu ne permettrait pas de les faire scroller au passage ?
Darks commented 3 months ago
Owner

Je vais voir comment régler le problème. :wink:

Je vais voir comment régler le problème. :wink:
Darks added the
bug
label 3 months ago
Darks self-assigned this 3 months ago
Darks commented 3 months ago
Owner

Bon, à priori c’est pas solvable sans utiliser de Javascript (source). Sauf que le menu doit être utilisable sans javascript.

Je laisse ouvert, mais ça sera pas traité en priorité.

J’ai eu une idée de workaround avec les @media :)

Bon, à priori c'est pas solvable sans utiliser de Javascript ([source](https://css-tricks.com/popping-hidden-overflow/)). Sauf que le menu doit être utilisable sans javascript. Je laisse ouvert, mais ça sera pas traité en priorité. J'ai eu une idée de workaround avec les `@media` :)
Darks added the
help wanted
label 3 months ago
Darks commented 2 weeks ago
Owner

Bon, si je passe le menu en ouvert par défaut, ça résout trivialement le problème :laughing:

Ça dérange ? Et puis ça évite le coup du :hover qui est pas spécialement recommandable

Bon, si je passe le menu en ouvert par défaut, ça résout trivialement le problème :laughing: Ça dérange ? Et puis ça évite le coup du `:hover` qui est pas [spécialement recommandable](https://doisjeutiliser.fr/unControleAuSurvol/)
Darks commented 2 weeks ago
Owner

Pour info, ça ressemblerait à ça. Le reste ne changeant absolument pas.

Vous pouvez checkout menu-opened pour essayer chez vous.

Si il n’y a pas d’avis défavorable exprimé d’ici lundi, je merge dans preprod.

Pour info, ça ressemblerait à ça. Le reste ne changeant absolument pas. Vous pouvez checkout `menu-opened` pour essayer chez vous. Si il n'y a pas d'avis défavorable exprimé d'ici lundi, je merge dans `preprod`.
Lephenixnoir commented 2 weeks ago
Owner

Hmm, ça me dérange pas particulièrement, c’est juste dommage parce que moins joli. L’équilibre icônes/labels semble pas évident à atteindre…

Quid de mettre chaque icône centrée avec le label en-dessous, pour réduire l’effet “largeur” du menu ?

Aussi, je suggère de centrer le logo Planète Casio dans tous les cas. :smiley:

Quoi que tu fasses, je suis d’accord pour merger.

Hmm, ça me dérange pas particulièrement, c'est juste dommage parce que moins joli. L'équilibre icônes/labels semble pas évident à atteindre... Quid de mettre chaque icône centrée avec le label en-dessous, pour réduire l'effet "largeur" du menu ? Aussi, je suggère de centrer le logo Planète Casio dans tous les cas. :smiley: Quoi que tu fasses, je suis d'accord pour merger.
Eragon commented 2 weeks ago
Collaborator

Centrer le logo de PC est une idée, pour le reste… je voit pas trop, je sais pas si c’est une bonne idée ou pas

Centrer le logo de PC est une idée, pour le reste... je voit pas trop, je sais pas si c'est une bonne idée ou pas
Darks commented 1 week ago
Owner

J’ai appliqué les modifs de Lephe dans le commit 7f63577c4f

Perso ça me va très bien comme ça, je merge dans dev puis preprod. Au pire on pourra rollback si ça vous va pas.

Comme ça on passe sur le VPS les dernières modifs visibles (index du forum entre autres)

J'ai appliqué les modifs de Lephe dans le commit 7f63577c4f Perso ça me va très bien comme ça, je merge dans `dev` puis `preprod`. Au pire on pourra rollback si ça vous va pas. Comme ça on passe sur le VPS les dernières modifs visibles (index du forum entre autres)
Lephenixnoir commented 1 week ago
Owner

C’est cool, merci ! Exactement ce que j’imaginais.

C'est cool, merci ! Exactement ce que j'imaginais.
Darks commented 1 week ago
Owner

J’ai légèrement modifié la taille du logo (un peu plus gros) et augmenté les marges entre éléments (+5 px). Pour le moment c’est en local, je pusherai avec la correction de #34

J'ai légèrement modifié la taille du logo (un peu plus gros) et augmenté les marges entre éléments (+5 px). Pour le moment c'est en local, je pusherai avec la correction de #34
Sign in to join this conversation.
No Milestone
No Assignees
4 Participants
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
Cancel
Save
There is no content yet.