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

Pas de :hover sur les catégories de la sidebar des tribunes #4396

Closed
A-312 opened this issue Jul 8, 2017 · 15 comments
Closed

Pas de :hover sur les catégories de la sidebar des tribunes #4396

A-312 opened this issue Jul 8, 2017 · 15 comments
Assignees
Labels
C-Front Concerne l'interface du site

Comments

@A-312
Copy link
Contributor

A-312 commented Jul 8, 2017

Bonjour,

Si on suit la logique des autres éléments, il devrait avoir un effet :hover sur les titres.

image

@artragis artragis added C-Front Concerne l'interface du site Facile Bon ticket pour débuter pour rejoindre le développement ! P-Basse labels Jul 25, 2017
@A-312
Copy link
Contributor Author

A-312 commented Aug 12, 2017

Avec la v25 les tutoriels c'est ok. Par conte les tribunes demande encore une modification.

@A-312
Copy link
Contributor Author

A-312 commented Aug 12, 2017

Je prends.

@A-312
Copy link
Contributor Author

A-312 commented Aug 12, 2017

Cet effet convient ?

@artragis
Copy link
Member

artragis commented Sep 2, 2017

@A-312 est-ce ok sur la v25 pour toi?

@motet-a
Copy link
Contributor

motet-a commented Sep 2, 2017

Le problème est toujours présent sur la bêta @artragis.

Notez que c’est plus cohérent avec le reste au niveau des marges et du padding en rajoutant (à l’arrache, faites pas ça en prod’) ces propriétés CSS :

    padding-top: 0;
    margin-top: 20px;
    line-height: 30px;
    background: white;

image

@A-312
Copy link
Contributor Author

A-312 commented Sep 4, 2017

Je n'ai pas la possibilité de faire le changement rapidement. J'avais fais une PR le jour même mais le rendu ne me convenait pas (si quelqu'un le fait avant, il n'y a pas de soucis).

@Situphen Situphen removed the P-Basse label Nov 11, 2017
@gllmc
Copy link
Member

gllmc commented Nov 21, 2017

Personnellement, l'absence d'effet :hover ne me choque pas. Par ailleurs, ces éléments ne se comportent pas comme les autres : ce ne sont pas des liens, ils n'affichent qu'un menu déroulant.

@A-312
Copy link
Contributor Author

A-312 commented Sep 6, 2018

Similaire à #4396 mais l'issue #4396 propose de rajouter un V.

Similaire à #932 mais l'issue #932 propose de rajouter un V.

@artragis
Copy link
Member

artragis commented Sep 6, 2018

tu as fait une erreur @A-312 on est déjà sur la 4396.

@A-312
Copy link
Contributor Author

A-312 commented Sep 6, 2018

Effectivement #932

@artragis
Copy link
Member

artragis commented Sep 6, 2018

d'ailleurs, on remarquera que @shiningfm a fait une remarque très pertinente :

Je pensais m'en occuper, mais en quoi les solutions proposées arrangent les choses en matière d'accessibilité ? Concrètement, rajouter une image ou un effet au survol, je ne trouve pas ça pertinent pour une personne aveugle (ou autre).

Il ne faudrait pas tout simplement spécifier des rôles WAI-ARIA ?

@A-312
Copy link
Contributor Author

A-312 commented Jan 30, 2019

Solution :

Faire comme le sommaire des tutoriels :

image

@Arnaud-D
Copy link
Contributor

Arnaud-D commented Apr 15, 2020

Avec l'arrivée de la bibliothèque, ce ticket semble obsolète. On ferme celui-là et on en refait un pour les billets ?

@A-312
Copy link
Contributor Author

A-312 commented Apr 15, 2020

Ca marche aussi pour les billets non ? 🤔

@A-312 A-312 changed the title Pas de :hover sur les catégories de la sidebar Pas de :hover sur les catégories de la sidebar des tribunes Apr 15, 2020
@Arnaud-D Arnaud-D removed the Facile Bon ticket pour débuter pour rejoindre le développement ! label Apr 21, 2020
@Situphen
Copy link
Member

Je ferme ici car j'ai décidé de partir sur un nouveau ticket pour plus de lisibilité. #6176

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
Projects
None yet
Development

No branches or pull requests

6 participants