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

Closed
opened 4 years ago by Ghost · 10 comments
Ghost commented 4 years 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.
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 4 years ago
Owner

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

Je vais voir comment régler le problème. :wink:
Darks added the bug label 4 years ago
Darks self-assigned this 4 years ago
Darks commented 4 years 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 4 years ago
Darks commented 3 years ago
Owner

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

Ç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 3 years 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`.
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. 😃

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 3 years 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 3 years 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)
Owner

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

C'est cool, merci ! Exactement ce que j'imaginais.
Darks commented 3 years 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
Darks closed this issue 3 years ago
Sign in to join this conversation.
No Milestone
No Assignees
4 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: devs/PCv5#27
Loading…
There is no content yet.