Il rapporto non è errato, ma la valutazione della gravità è un po’ imprecisa. Questi link non sono contenuti generati dagli utenti. Solo gli amministratori possono aggiungerli tramite la dashboard di amministrazione. Quindi, è ragionevole supporre che, in qualità di amministratore, non collegherai siti di cui non ti fidi.
Detto questo, ho avuto una breve conversazione con David, che ha suggerito di aggiungere comunque rel="noopener" ai link esterni. L’ho aggiunto qui. Se un link è impostato per aprirsi in una nuova scheda, viene aggiunto automaticamente.
Per quanto riguarda noreferrer,
ritengo che sia più vantaggioso non aggiungerlo, poiché potresti voler collegare il tuo sito principale e ottenere analisi su quanti utenti vengono reindirizzati al sito principale dalla tua community.
È ancora così? Puoi condividere uno screenshot delle impostazioni del tuo componente?
Sì, l’ho notato anch’io quando ho aggiornato, ma non è correlato alle nuove modifiche. In qualche modo il nuovo impostazione non viene rilevata immediatamente dal compilatore. Quindi, potresti dover spuntare quell’impostazione e poi deselezionarla.
Nessun problema, però: dai miei test, l’errore non ha alcun impatto sul tuo sito.
Il nostro sito mi ha mostrato oggi che dovrei aggiornare il componente del tema, cosa che ho appena fatto cliccando sul pulsante Aggiorna.
Ma ora mostra l’errore: Errore: Variabile non definita: "$add-whitespace". alla riga 14 di common.scss >> @if $add_whitespace == "true" { ----^
Siamo a conoscenza di questo problema, ma non dovrebbe avere alcun impatto sul tuo sito. Si tratta di un problema del compilatore che stiamo monitorando internamente.
Fammi sapere se causa problemi sul tuo sito: non dovrebbe farlo.
Vero, non ho notato ulteriori problemi.
Sto attualmente eseguendo una ricostruzione di Discourse del sito e controllerò di nuovo dopo. Se l’errore sarà ancora presente, controllerò il suggerimento di Jonathan.
dopo la ricostruzione l’errore è sparito
Grazie per il tuo ottimo componente del tema che utilizziamo già da due anni.
Ciao, sto utilizzando Discourse 2.8.0.beta1 e quando provo ad aggiornare all’ultima versione di questo componente del tema, ricevo questo messaggio di errore:
Errore: Variabile non definita: "$add-whitespace". alla riga 14 di common.scss >> @if $add_whitespace == "true" { ----^
Ciao, grazie per questo plugin.
Ho fatto alcune osservazioni riguardanti le icone FA che vorrei condividere:
L’integrazione delle icone far- sembra funzionare (regular), ma quella delle fas- no (solid).
Puoi provare questo esempio:
fas-building
vs.
far-building
Hai qualche idea al riguardo? Le icone far- sono poche rispetto alle fas- (a parte la prospettiva di design), quindi questo è un problema…
Penso che tu debba cercare nel codice per vedere quali icone possono essere utilizzate e quali nomi assegnare loro (a meno che non sia stato modificato dal commento sopra).
Beh, la descrizione del componente del tema suggerisce che si possa utilizzare ogni icona FA5 gratuita, indipendentemente dalla sua classe (brand, regular, solid).
Dai un’occhiata al codice a cui hai fatto riferimento:
D’accordo - in questo caso, dovrebbe essere corretto nella descrizione del componente del tema? Ma in realtà funziona su tutte le icone far-, quindi la mia opinione è che le fas- siano state semplicemente “dimenticate”.
Funziona per me.. è inclusa di default una sottoinsieme (puoi vederlo in /styleguide/atoms/icons). Gli icon aggiuntivi devi prima aggiungerli, sia nelle impostazioni di amministrazione che direttamente nel componente, come in questo caso:
Ce l’ho fatta funzionare anch’io (usando “building” nelle icone SVG e nel componente del tema) – mostra la versione solida dell’icona Potrei dover ricredermi (sono ancora quasi sicuro che alcune cose non funzionassero quando avrebbero dovuto…)