Liens d'en-tête personnalisés (icônes)

J’ai effectué une mise à jour et je vois le message d’erreur suivant sur la page /admin/customize/themes :

Erreur : Variable non définie : "$add-whitespace". à la ligne 14 de common.scss >> @if $add_whitespace == "true" { ----^

Mise à jour : J’ai coché la nouvelle case « whitespace », puis je l’ai décochée (en sauvegardant à chaque fois), et l’erreur a disparu.

2 « J'aime »

Le rapport n’est pas erroné, mais l’évaluation de la gravité est un peu inexacte. Ces liens ne sont pas du contenu généré par les utilisateurs. Seuls les administrateurs peuvent les ajouter via le tableau de bord d’administration. Il est donc raisonnable de supposer qu’un administrateur ne liera pas des sites qu’il ne fait pas confiance.

Cela dit, j’ai eu une brève discussion avec David, et il a suggéré d’ajouter rel="noopener" aux liens externes de toute façon. Je l’ai ajouté ici. Si un lien est configuré pour s’ouvrir dans un nouvel onglet, il est ajouté automatiquement.

Concernant noreferrer,

Je pense qu’il est plus avantageux de ne pas l’ajouter, car vous pourriez vouloir lier votre site principal et obtenir des analyses sur le nombre d’utilisateurs redirigés vers le site principal depuis votre communauté.

Est-ce toujours le cas ? Pouvez-vous partager une capture d’écran de vos paramètres de composant ?

Oui, j’ai remarqué cela lors de ma mise à jour également, mais cela n’est pas lié aux nouvelles modifications. D’une manière ou d’une autre, le nouveau paramètre n’est pas pris en compte immédiatement par le compilateur. Vous devrez peut-être cocher ce paramètre et le décocher.

Pas de souci cependant, l’erreur n’a aucun impact sur votre site selon mes tests.

5 « J'aime »

Notre site m’a indiqué aujourd’hui que je devrais mettre à jour le composant de thème, ce que j’ai fait en cliquant sur le bouton de mise à jour.
Mais maintenant, l’erreur suivante s’affiche :
Erreur : Variable indéfinie : "$add-whitespace". sur la ligne 14 de common.scss >> @if $add_whitespace == "true" { ----^

J’ai eu ce problème et il s’est résolu tout seul lorsque j’ai coché puis décoché la case : Custom Header Links (icons) - #129 by Jonathan5

2 « J'aime »

Merci pour le rapport @GuidoD

Nous sommes au courant de ce problème, mais il ne devrait pas avoir d’impact sur votre site. Il s’agit d’un problème de compilateur que nous suivons en interne.

Faites-moi savoir si cela cause des problèmes sur votre site - ce ne devrait pas être le cas.

2 « J'aime »

C’est vrai, je n’ai remarqué aucun autre problème.
Je suis actuellement en train de reconstruire le site avec Discourse et je vérifierai à nouveau après. Si l’erreur persiste, je suivrai l’indice de Jonathan.

Après la reconstruction, l’erreur a disparu :slight_smile:

Merci pour ton excellent composant de thème que nous utilisons déjà depuis deux ans.

2 « J'aime »

Bonjour, je suis sur Discourse 2.8.0.beta1 et lors de la mise à jour vers la dernière version de ce composant de thème, j’obtiens ce message d’erreur :

Erreur : Variable indéfinie : "$add-whitespace". à la ligne 14 de common.scss >> @if $add_whitespace == "true" { ----^
2 « J'aime »

Je rencontre la même erreur. Je suis sur la version stable 2.7.

Je comprends. Comme je l’ai mentionné ci-dessus, cela n’aura aucun impact sur votre site.

4 « J'aime »

Je peux confirmer que cela fonctionne.

Cochez-le, puis décochez-le et rechargez la page.

screenshot-vapingcommunity.co.uk-2021.06.07-13_48_21

Bonjour, merci pour ce plugin.
J’ai fait quelques observations concernant les icônes FA que je souhaite partager :
L’intégration des icônes far- semble fonctionner (regular), mais pas celle des fas- (solid).

Vous pouvez essayer ceci :

fas-building
contre
far-building

Qu’en pensez-vous ? Il y a peu d’icônes far- par rapport aux fas- (sans même parler de l’aspect design), ce qui pose donc un problème…

Je pense que vous devez parcourir le code pour voir quelles icônes peuvent être utilisées et quels noms leur attribuer (sauf si cela a été modifié depuis le commentaire ci-dessus).

Eh bien, la description du composant de thème suggère que l’on peut utiliser chaque icône FA5 gratuite, indépendamment de sa classe (brand, regular, solid).

Jetez un œil au code auquel vous faites référence :

Il n’y a pas de clause if pour solid. Donc, cela devrait être écrit ainsi :

when "solid"
  key = "fas-"

?

À ma connaissance, cette affirmation a toujours été erronée.

D’accord – dans ce cas, cela devrait être corrigé dans la description du composant de thème ?
Mais en réalité, cela fonctionne bien avec toutes les icônes far-, donc mon avis est que les fas- ont simplement été « oubliés ».

1 « J'aime »

La limite de mes connaissances est que cela ne fonctionne pas comme décrit, et comment savoir quels icônes fonctionneront en examinant le code :slight_smile:

Je pense que les noms de variables pour les icônes solides sont généralement implémentés sans préfixe. Avez-vous simplement essayé building ?

Oui, bien sûr. Sans succès.
Je suppose qu’un petit sous-ensemble de fas- est livré avec Discourse.

Ça fonctionne de mon côté… un sous-ensemble est inclus par défaut (vous pouvez le voir dans /styleguide/atoms/icons). Pour les icônes supplémentaires, vous devez d’abord les ajouter, soit dans les paramètres d’administration, soit directement dans le composant, comme dans ce cas :

2 « J'aime »

J’ai réussi à faire fonctionner cela aussi (en utilisant “building” dans les icônes SVG et dans le composant de thème) — cela affiche la version pleine de l’icône :+1: Il se peut que je doive manger mes mots (je suis toujours presque certain que certaines choses ne fonctionnaient pas alors qu’elles auraient dû…)

2 « J'aime »