Corrige le style et la semantique lors de la desinscription

[mehdbnaleg]

Traite l'issue #5716

J'ai ajouté une balise <samp> pour améliorer la sémantique de la page, et modifié le style de la balise <kbd>.

Q/A

• Se connecter comme utilisateur lambda
• Se rendre dans les Paramètres -> Désinscription

Je m'y suis prise comme un bourrin avec le style, mais ça marche. Je ne sais pas si ça risque de casser quelque chose dans le reste du site, mais vu que c'est juste une modification du style de <kbd> je pense que les chances sont faibles.

Je souhaiterai avoir une couleur plus rougeâtre pour le texte, pour qu'il puisse bien correspondre au bouton auquel il fait référence, vous en pensez quoi ?

Merci pour votre temps !

[coveralls]

Coverage remained the same at 86.859% when pulling 52ae0f3 on ToxicScorpius:issue/5716 into 6b84ea1 on zestedesavoir:dev.

[AmauryCarrade]

Merci ! J'ai un commentaire sur ta façon de gérer les styles, afin d'éviter de la duplication de code.

Je m'y suis prise comme un bourrin avec le style, mais ça marche. Je ne sais pas si ça risque de casser quelque chose dans le reste du site, mais vu que c'est juste une modification du style de <kbd> je pense que les chances sont faibles.

Avec le CSS qu'on a, on ne sait jamais. :p

Je souhaiterai avoir une couleur plus rougeâtre pour le texte, pour qu'il puisse bien correspondre au bouton auquel il fait référence, vous en pensez quoi ?

On pourrait, il ne faudrait juste pas qu'on confonde avec un bouton mais il y a peu de risque je pense. Le plus simple, si tu suis ce que je propose plus bas, c'est de modifier le style des éléments kbd existants pour ajouter une variation via une classe supplémentaire :

kbd {
    // code existant

    &.is-red {
        // Code qui change sa couleur (le reste étant inchangé)
    }
}

…et ajouter la classe à l'élément kbd utilisé ici.

<kbd class="is-red"><samp>…</samp></kbd>

Pour la couleur, tu peux en piocher une ici. Le plus simple si tu veux changer ça c'est d'attendre que la PR normes soit fusionnée, et de rebaser sur dev après : tu auras accès à toutes les variables SCSS des couleurs, etc.

[AmauryCarrade]

J'ai pas QA mais ça me semble pas trop mal. Par contre, il va falloir attendre #5922 pour pouvoir intégrer les couleurs et autres en tant que variables, et éventuellement utiliser un .content générique s'il y en a un d'ici là (sinon pas grave ça peut attendre ça).

[Situphen]

@ToxicScorpius Il y a des conflits à résoudre par rapport à la branche upstream/dev étant donné que la PR qui met en place Black a été fusionnée et à modifié le style des fichiers Python. :)

[AmauryCarrade]

Quand tu reprendras cette PR, il faudra rebaser pour inclure les modifications de la norme, puis appliquer cette dernière :) .

[Arnaud-D]

Je repasse à "En attente de QA". J'ai essayé de la faire, mais je me suis embrouillé et je lâche l'affaire pour le moment. :)

[Arnaud-D]

QA NOK ❌

Alors on peut faire le rebase sans trop de problèmes, mais le bouton rouge foire (Firefox, dernière version) ; je ne vois pas trop pourquoi cependant, au vu des modifs...

[Arnaud-D]

Reprise par #6396.