Olá @meghna, fiz um pull request com alguns bugs corrigidos.
Obrigado @Canapin, o PR foi mesclado. ![]()
Obrigado por este ótimo componente de tema.
Você notou que a imagem de decoração aparece muito cedo quando a barra lateral está ativa? Parece que ela aparece com 1450px de largura. Por volta de 1700px seria melhor com a barra lateral. Não sei se isso é possível.
Eu agora uso este componente em vez das minhas próprias personalizações, embora eu prefira lâmpadas arredondadas ![]()
No entanto, ainda há um pequeno problema com o componente. As bases das lâmpadas ficam escondidas atrás do cabeçalho:
Eu fiz um PR para corrigir isso.
Para aqueles que preferem lâmpadas arredondadas, você pode adicionar este CSS ao seu próprio tema:
.lightrope li {
width: 12px;
height: 15px;
}
}
Obrigado @Canapin, eu mesclei o PR. ![]()
Bom ponto! Vou dar uma olhada.
editar: Fiz um PR.
A posição vertical ficará incorreta se você usar este tema junto com o componente de tema Welcome Banner, mesmo que o banner esteja oculto.
Veja:
https://meta.discourse.org/t/welcome-link-banner/218743/61?u=canapin
A maneira mais direta de contornar o problema é adicionar este CSS ao seu tema:
.above-main-container-outlet.welcome-link-banner-connectors {
margin-bottom: 0;
}
Atualizei o componente do banner de link de boas-vindas para evitar isso. Obrigado novamente @Canapin!
Hoje houve uma atualização para este componente, então eu o atualizei, mesmo que esteja desativado aqui, pois o Natal já acabou. Agora vejo o seguinte:
Erro: CSS inválido após "..."hadow: 0px math": expressão esperada (por exemplo, 1px, negrito), foi ".div($globe-height," na linha 44 de ../../../tmp/discourse_theme_acdcea7663ae5fb94fa2525e54768195/discourse-christmas-decorations/stylesheets/lights.scss da linha 2 de common.scss >> box-shadow: 0px math.div($globe-height, 6) $globe-width*2 $globe-sprea --------------------------^
Algo quebrou agora. ![]()
Você está usando a versão mais recente do Discourse?
Eu executo 3.1.0.beta2 [e64d1c4105] aqui. Então, versão Beta sim, mas certamente não o último commit.
O mesmo com [7eb6223b04] ![]()
Reverti o commit por enquanto e trabalharei com o @Falco para um conserto adequado.
Estava testando isso e descobri que os chapéus não aparecem na visualização móvel.
@meghna
obrigado por esta coisa legal!
uma sugestão, os sinos dos dois lados deveriam ser coloridos, agora estão em preto/branco, não tão festivos? haha, você decide…
na visualização móvel, não tem a lista de contribuidores, se conseguir um chapéu para cada autor de tópico, isso parece muito ocupado,
parece bom agora no celular, minha opinião.
Eu escolhi o preto/branco para que fique bom em todos os tipos de esquemas de cores. PR é bem-vindo ![]()
outra sugestão,
seria bom se pudesse adicionar uma configuração para alterar a imagem do banner de luz, talvez várias opções, ou alterar aleatoriamente.
por exemplo, ativar este componente agora, ainda faltam várias semanas para o Natal, seria bom se pudesse haver algumas mudanças… ![]()
Você pode consertar os chapéus para celular? ![]()



