Componente decorazione di Natale 🎄

Ciao @meghna, ho creato una pull request con alcune correzioni di bug.

5 Mi Piace

Grazie @Canapin, PR è stato unito. :+1:

4 Mi Piace

Grazie per questo ottimo componente tematico.

Hai notato che l’immagine decorativa appare troppo presto quando la barra laterale è attiva? Sembra apparire a una larghezza di 1450px. Intorno a 1700px potrebbe essere meglio con la barra laterale. Non so se sia possibile.

2 Mi Piace

Ora uso questo componente invece delle mie personalizzazioni, anche se preferisco le lampadine arrotondate :blush:

Tuttavia, c’è ancora un piccolo problema con il componente. Le basi delle lampadine sono nascoste dietro l’intestazione:

Ho creato una PR per risolvere il problema.


Per coloro che preferiscono le lampadine arrotondate, è possibile aggiungere questo CSS al proprio tema:

.lightrope li {
        width: 12px;
        height: 15px;
    }
}

5 Mi Piace

Grazie @Canapin, ho unito la PR. :heart:

5 Mi Piace

È normale che con gli smartphone sia così basso?

1 Mi Piace

Ottima osservazione! Ci darò un’occhiata.


edit: Ho creato una PR.

3 Mi Piace

La posizione verticale sarà errata se si utilizza questo tema insieme al componente tema Banner di benvenuto anche se il banner è nascosto.

Vedi:

https://meta.discourse.org/t/welcome-link-banner/218743/61?u=canapin


Il modo più semplice per aggirare il problema è aggiungere questo CSS al tuo tema:

.above-main-container-outlet.welcome-link-banner-connectors {
    margin-bottom: 0;
}
3 Mi Piace

Ho aggiornato il componente del banner del link di benvenuto per evitare questo. Grazie ancora @Canapin!

3 Mi Piace

Oggi c’è stato un aggiornamento per questo componente, quindi l’ho aggiornato, anche se qui è disabilitato poiché il Natale è finito. Ora vedo quanto segue:

Errore: CSS non valido dopo \"...hadow: 0px math\": previsto espressione (es. 1px, grassetto), era \".div($globe-height,\" alla riga 44 di ../../../tmp/discourse_theme_acdcea7663ae5fb94fa2525e54768195/discourse-christmas-decorations/stylesheets/lights.scss dalla riga 2 di common.scss >> box-shadow: 0px math.div($globe-height, 6) $globe-width*2 $globe-sprea --------------------------^

Qualcosa si è rotto adesso. :wink:

1 Mi Piace

Sei sull’ultima versione di Discourse?

1 Mi Piace

Eseguo 3.1.0.beta2 [e64d1c4105] qui. Quindi, versione Beta sì, ma sicuramente non l’ultimo commit.

1 Mi Piace

Uguale con [7eb6223b04] :wink:

1 Mi Piace

Ho annullato il commit per ora e lavorerò con @Falco per una corretta correzione.

1 Mi Piace

Stavo testando questo e ho scoperto che i cappelli non vengono visualizzati nella visualizzazione mobile.

@meghna
grazie per questo bel pezzo!

un suggerimento, le campane su entrambi i lati dovrebbero essere colorate, ora sono in bianco e nero, non molto festose? haha, decidi tu…

1 Mi Piace

Nella vista mobile, non ha l’elenco dei collaboratori, se ottiene un cappello per ogni autore dell’argomento, sembra troppo affollato,
sembra a posto ora su mobile, i miei 2 centesimi.

Ho scelto il bianco/nero in modo che si abbini a tutti i tipi di schemi di colori. PR ben accetto :slight_smile:

3 Mi Piace

un altro suggerimento,
sarebbe bello se si potesse aggiungere un’impostazione per cambiare l’immagine del banner luminoso, magari diverse opzioni, o cambiarla casualmente.
diciamo, attiva questo componente ora, mancano ancora diverse settimane a Natale, è bello se ci potessero essere dei cambiamenti… :slight_smile:

1 Mi Piace

Puoi sistemare i cappelli per mobile? :grinning:

2 Mi Piace