Link di intestazione personalizzati (icone)

Ho aggiornato questa versione e ricevo il seguente messaggio di errore nella pagina /admin/customize/themes:

Errore: Variabile non definita: "$add-whitespace". alla riga 14 di common.scss >> @if $add_whitespace == "true" { ----^

Aggiornamento: Ho spuntato la nuova casella “whitespace” e poi deselezionata (salvando ogni volta) e l’errore è scomparso.

2 Mi Piace

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.

5 Mi Piace

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" { ----^

Avevo questo problema e si è risolto da solo quando ho spuntato e deselezionato la casella: Custom Header Links (icons) - #129 by Jonathan5

2 Mi Piace

Grazie per la segnalazione @GuidoD

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.

2 Mi Piace

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 :slight_smile:

Grazie per il tuo ottimo componente del tema che utilizziamo già da due anni.

2 Mi Piace

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" { ----^
2 Mi Piace

Ho lo stesso errore anch’io. Sono sulla versione stabile 2.7.

Capisco. Come ho già menzionato sopra, questo non avrà alcun impatto sul tuo sito.

4 Mi Piace

Posso confermare che questo funziona.

Seleziona questa opzione, poi deselezionala e ricarica la pagina

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

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:

Non c’è nessun if-clause per solid. Quindi dovrebbe essere scritto così:

when "solid"
  key = "fas-"

?

Per quanto ne so, quella affermazione è sempre stata errata.

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”.

1 Mi Piace

Il limite delle mie conoscenze è che non funziona come descritto, e come capire quali icone funzionano guardando il codice :slight_smile:

Penso che i nomi delle variabili per le icone solide siano generalmente implementati senza prefisso. Hai provato semplicemente building?

Sì, certo. Senza successo.
Immagino che solo un piccolo sottoinsieme di fas- sia incluso in Discourse.

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:

2 Mi Piace

Ce l’ho fatta funzionare anch’io (usando “building” nelle icone SVG e nel componente del tema) – mostra la versione solida dell’icona :+1: Potrei dover ricredermi (sono ancora quasi sicuro che alcune cose non funzionassero quando avrebbero dovuto…)

2 Mi Piace