Projet

Général

Profil

Bug #40826

supprimer déclaration css .sr-only de .gru-nav-button

Ajouté par Thomas Jund (congés, retour le 29/04) il y a environ 4 ans. Mis à jour il y a environ 4 ans.

Statut:
Fermé
Priorité:
Normal
Version cible:
-
Début:
19 mars 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Suite à #36909 où `.sr_only` est maintenant une class utilitaire globale, on peut supprimer la déclaration spécifique pour .gru-nav-button


Fichiers


Demandes liées

Lié à Intégrations graphiques Publik - Development #36909: New utils mixin & class: sr-only (screen-reader-only)Fermé14 octobre 2019

Actions

Révisions associées

Révision 4629ba29 (diff)
Ajouté par Thomas Jund (congés, retour le 29/04) il y a environ 4 ans

scss: remove .gru-nav-button .sr-only declaration (#40826)

Historique

#1

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 4 ans

  • Lié à Development #36909: New utils mixin & class: sr-only (screen-reader-only) ajouté
#2

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 4 ans

#3

Mis à jour par Nicolas Roche il y a environ 4 ans

cf #36909, "lisible par les lecteurs l'écran et les bots" : j'aurais bien voulu tester le rendu de sr-only.
Est-ce que l'on parle de ça : https://support.mozilla.org/en-US/kb/firefox-reader-view-clutter-free-web-pages ?
Parce que l'icone "Reader View" ne s'affiche pas avec Publik (bien qu'elle s'affiche avec redmine).

#4

Mis à jour par Frédéric Péters il y a environ 4 ans

Est-ce que l'on parle de ça : https://support.mozilla.org/en-US/kb/firefox-reader-view-clutter-free-web-pages ?

Non on parle des lecteurs d'écran pour aveugles/malvoyants etc. sr-only cf #36909, "the industry accepted way to achieve a screenreader only element".

#5

Mis à jour par Nicolas Roche il y a environ 4 ans

  • Statut changé de Solution proposée à Solution validée

Je ne constate aucune régression, car "display: none" est par ailleurs posé plus loin :

.gru-nav-button {
    display: none;
}

#6

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 4629ba29542aa5478bdc1016383eed4530e8225a (HEAD -> master, origin/master, origin/HEAD)
Author: Thomas JUND <tjund@entrouvert.com>
Date:   Thu Mar 19 11:20:48 2020 +0100

    scss: remove .gru-nav-button .sr-only declaration (#40826)
#7

Mis à jour par Frédéric Péters il y a environ 4 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF