Componente de Decoração de Natal 🎄

Olá @meghna, fiz um pull request com alguns bugs corrigidos.

5 curtidas

Obrigado @Canapin, o PR foi mesclado. :+1:

4 curtidas

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.

2 curtidas

Eu agora uso este componente em vez das minhas próprias personalizações, embora eu prefira lâmpadas arredondadas :blush:

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;
    }
}

5 curtidas

Obrigado @Canapin, eu mesclei o PR. :heart:

5 curtidas

É normal que com smartphones seja tão baixo?

1 curtida

Bom ponto! Vou dar uma olhada.


editar: Fiz um PR.

3 curtidas

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;
}
3 curtidas

Atualizei o componente do banner de link de boas-vindas para evitar isso. Obrigado novamente @Canapin!

3 curtidas

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. :wink:

1 curtida

Você está usando a versão mais recente do Discourse?

1 curtida

Eu executo 3.1.0.beta2 [e64d1c4105] aqui. Então, versão Beta sim, mas certamente não o último commit.

1 curtida

O mesmo com [7eb6223b04] :wink:

1 curtida

Reverti o commit por enquanto e trabalharei com o @Falco para um conserto adequado.

1 curtida

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…

1 curtida

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

3 curtidas

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

1 curtida

Você pode consertar os chapéus para celular? :grinning:

2 curtidas