Links de cabeçalho personalizados (ícones)

Atualizei isso e vejo a seguinte mensagem de erro na página /admin/customize/themes:

Erro: Variável indefinida: "$add-whitespace". na linha 14 de common.scss >> @if $add_whitespace == "true" { ----^

Atualização: Marquei a nova caixa de seleção “whitespace” e desmarquei (salvando a cada vez) e o erro desapareceu.

2 curtidas

O relatório não está errado, mas a avaliação de gravidade está um pouco fora do alvo. Esses links não são conteúdo gerado pelo usuário. Apenas administradores podem adicioná-los por meio do painel de administração. Portanto, é seguro assumir que, como administrador, você não vinculará sites em que não confia.

Dito isso, conversei rapidamente com o David, e ele sugeriu adicionar rel="noopener" aos links externos de qualquer maneira. Fiz isso aqui. Se um link estiver configurado para abrir em uma nova aba, ele será adicionado automaticamente.

Quanto ao noreferrer,

Acho que é mais benéfico não adicioná-lo, pois você pode querer vincular seu site principal e obter análises sobre quantos usuários são encaminhados ao site principal a partir da sua comunidade.

Isso ainda está acontecendo? Você pode compartilhar uma captura de tela das configurações do seu componente?

Sim, eu vi isso também quando atualizei, mas não está relacionado às novas alterações. De alguma forma, a nova configuração não está sendo detectada pelo compilador imediatamente. Então, talvez você precise marcar e desmarcar essa configuração.

Sem problemas, porém; pelo que testei, o erro não tem impacto algum no seu site.

5 curtidas

Nosso site me mostrou hoje que eu deveria atualizar o componente do tema, o que acabei de fazer clicando no botão de atualização.
Mas agora ele exibe o erro:
Erro: Variável indefinida: "$add-whitespace". na linha 14 de common.scss >> @if $add_whitespace == "true" { ----^

Eu tive isso e se resolveu quando marquei e desmarquei a caixa: Custom Header Links (icons) - #129 by Jonathan5

2 curtidas

Obrigado pelo relatório, @GuidoD.

Estamos cientes desse problema, mas ele não deve ter qualquer impacto no seu site. Trata-se de um problema de compilador que estamos acompanhando internamente.

Avise-me se isso causar algum problema no seu site — o que não deve acontecer.

2 curtidas

Verdade, não notei nenhum problema adicional.
Atualmente estou fazendo uma reconstrução do Discourse do site e vou verificar novamente depois. Se o erro ainda estiver lá, vou verificar a sugestão do Jonathan.

Depois da reconstrução, o erro desapareceu :slight_smile:

Obrigado pelo seu ótimo componente de tema que já usamos há dois anos.

2 curtidas

Olá, estou na versão Discourse 2.8.0.beta1 e, ao atualizar para a última versão deste componente de tema, recebo a seguinte mensagem de erro:

Erro: Variável indefinida: "$add-whitespace". na linha 14 de common.scss >> @if $add_whitespace == "true" { ----^
2 curtidas

Eu também tenho o mesmo erro. Estou na versão 2.7 estável.

Entendo. Como mencionei acima, isso não afetará seu site de forma alguma.

4 curtidas

Posso confirmar que isso funciona.

Marque e depois desmarque essa opção e recarregue a página.

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

Olá, obrigado por este plugin.
Fiz algumas observações sobre os ícones FA que gostaria de compartilhar:
A integração dos ícones far- parece funcionar (regular), mas a dos fas- não (solid).

Você pode testar este aqui:

fas-building
vs.
far-building

O que você acha disso? Existem poucos ícones far- em comparação aos fas- (além da perspectiva de design), então isso é um problema…

Acho que você precisa pesquisar no código para ver quais ícones podem ser usados e quais nomes usar para eles (a menos que tenha sido alterado desde o comentário acima).

Bem, a descrição do componente do tema sugere que é possível usar qualquer ícone gratuito do FA5, independentemente de sua classe (brand, regular, solid).

Dê uma olhada no código a que você se referiu:

Não há nenhuma cláusula if para solid. Então deveria estar escrito:

when "solid"
  key = "fas-"

?

Pelo que pude constatar, essa alegação sempre esteve errada.

Tudo bem - neste caso, deveria ser corrigido na descrição do componente do tema?
Mas na verdade ele funciona em todos os ícones far-, então minha opinião é que os fas- foram apenas “esquecidos”.

1 curtida

O limite do meu conhecimento é que não está funcionando conforme descrito, e como descobrir quais ícones funcionarão olhando o código :slight_smile:

Acho que os nomes das variáveis para os ícones sólidos geralmente são implementados sem prefixo. Você tentou apenas building?

Sim, claro. Sem sucesso.
Acho que apenas um pequeno subconjunto de fas- é distribuído com o Discourse.

Funciona para mim… há um subconjunto incluído por padrão (você pode ver isso em /styleguide/atoms/icons). Ícones adicionais você precisa adicionar primeiro, seja nas configurações de administrador ou no próprio componente, como neste caso:

2 curtidas

Consegui fazer isso funcionar também (usando ‘building’ nos ícones SVG e no componente do tema) — mostra a versão sólida do ícone :+1: Talvez eu tenha que comer minhas palavras (ainda estou quase certo de que algumas coisas não funcionaram como deveriam…)

2 curtidas