Petite demande de fonctionnalité pour core : ajouter 3 classes

Bonjour, je travaille sur le style de la boîte de citation… Est-il possible que le cœur de Discourse ajoute des classes à la flèche vers le bas (voir la citation dans son contexte) et à la flèche vers le haut (aller au message cité) ?

Et — plus important encore — pourrait-on également ajouter une classe sur l’élément .quote aside principal (div parent) pour indiquer si l’utilisateur cité est bloqué ?

Je ne demande aucun changement visuel, ni aucune modification du fonctionnement de la fonction « ignorer » :wink: Je souhaite simplement disposer de certaines classes afin d’avoir des options à ma disposition et pour les futurs développeurs de thèmes.

Je suis également prêt à essayer cela moi-même et à soumettre une PR, si quelqu’un peut simplement m’indiquer grossièrement la bonne direction pour savoir où dans la base de code je devrais tenter de le faire.

Êtes-vous sûr d’en avoir besoin ? Ne pouvez-vous pas cibler cela d’autres manières ? Vous pouvez même utiliser l’attribut title pour le cibler si nécessaire.

Je pourrais peut-être cibler ces deux éléments (les flèches) en CSS, oui. Cela nécessitera des sélecteurs un peu complexes et sera évidemment plus fragile que si nous ajoutions directement des classes sur ces éléments, mais je devrais y arriver.

Le plus difficile est de déterminer si cet utilisateur est ignoré ou non. D’après ce que je vois, aucun des conteneurs parents n’ajoute de classe pour indiquer le statut « ignoré » pour la citation. Sans cette classe, je ne peux rien faire de différent en CSS. Qu’en penses-tu ? Dis-moi si je me trompe :slight_smile:

L’utilisateur ignoré a été cité, il n’est probablement pas là ; cela nécessiterait en effet une modification du cœur.

Je ne sais pas trop comment je me sens à l’idée de faire ce travail pour ajouter la classe. Je suppose que si vous soumettez une PR et que c’est assez simple, je suis d’accord pour la fusionner.

Je vais voir si je peux comprendre :slight_smile:

C’était étonnamment facile ! J’ai mis plus de temps à trouver où ajouter le code qu’à écrire cette seule ligne. N’est-ce pas toujours comme ça ? :wink:

HÉ LA TEAM, C’EST MA PREMIÈRE DEMANDE DE PULL POUR LE CŒUR DE DISCOURSE ! :partying_face:

PR en place : FEATURE: add class to <aside> quote block when quoting an ignored user by pnoeric · Pull Request #9511 · discourse/discourse · GitHub

Les flèches haut et bas sont simples

.quote-controls .svg-icon-title
.quote-controls .back

Je ne suis pas du tout sûr concernant les publications bloquées ; vous pourriez avoir raison en disant que votre PR est la meilleure solution.