Composant de décoration de Noël 🎄

Salut @meghna, j’ai fait une pull request avec quelques bugs corrigés.

5 « J'aime »

Merci @Canapin, la PR est fusionnée. :+1:

4 « J'aime »

Merci pour ce super composant de thème.

Avez-vous remarqué que l’image de décoration apparaît trop tôt lorsque la barre latérale est activée ? Elle semble apparaître à une largeur de 1450px. Environ 1700px serait peut-être mieux avec la barre latérale. Je ne sais pas si c’est possible.

2 « J'aime »

J’utilise maintenant ce composant au lieu de mes propres personnalisations, bien que je préfère les ampoules rondes :blush:

Cependant, il y a toujours un léger problème avec le composant. Les bases des ampoules sont cachées derrière l’en-tête :

J’ai fait une PR pour corriger cela.


Pour ceux qui préfèrent les ampoules rondes, vous pouvez ajouter ce CSS à votre propre thème :

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

5 « J'aime »

Merci @Canapin, j’ai fusionné la PR. :heart:

5 « J'aime »

Est-ce normal que ce soit si bas avec les smartphones ?

1 « J'aime »

Bien vu ! Je vais examiner ça.


edit : J’ai créé une PR.

3 « J'aime »

La position verticale sera incorrecte si vous utilisez ce thème avec le composant de thème Welcome Banner, même si la bannière est masquée.

Voir :

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


La manière la plus simple de contourner le problème est d’ajouter ce CSS à votre thème :

.above-main-container-outlet.welcome-link-banner-connectors {
    margin-bottom: 0;
}
3 « J'aime »

J’ai mis à jour le composant de la bannière de lien de bienvenue pour éviter cela. Merci encore @Canapin !

3 « J'aime »

Il y a eu une mise à jour pour ce composant aujourd’hui, je l’ai donc mis à jour, même s’il est désactivé ici car Noël est terminé. Maintenant, je vois ceci :

Erreur : CSS invalide après « ...hadow: 0px math » : expression attendue (par exemple, 1px, bold), mais trouvé « .div($globe-height,» à la ligne 44 de ../../../tmp/discourse_theme_acdcea7663ae5fb94fa2525e54768195/discourse-christmas-decorations/stylesheets/lights.scss à partir de la ligne 2 de common.scss >> box-shadow: 0px math.div($globe-height, 6) $globe-width*2 $globe-sprea --------------------------^

Quelque chose est cassé maintenant. :wink:

1 « J'aime »

Êtes-vous sur la dernière version de Discourse ?

1 « J'aime »

J’exécute 3.1.0.beta2 [e64d1c4105] ici. Donc, version bêta oui, mais certainement pas le dernier commit.

1 « J'aime »

Idem avec [7eb6223b04] :wink:

1 « J'aime »

J’ai annulé le commit pour le moment et je travaillerai avec @Falco pour une correction appropriée.

1 « J'aime »

J’effectuais des tests et j’ai constaté que les chapeaux ne s’affichaient pas en mode mobile.

@meghna
merci pour ce joli !

une suggestion, les cloches des deux côtés devraient-elles être colorées, elles sont noires/blanches maintenant, pas très festives ? haha, à vous de voir…

1 « J'aime »

En vue mobile, il n’y a pas la liste des contributeurs, si on obtient un chapeau pour chaque auteur de sujet, ça a l’air trop chargé, ça semble correct maintenant sur mobile, mon avis.

J’ai choisi le noir/blanc pour qu’il s’accorde bien avec tous les types de palettes de couleurs. Une PR est la bienvenue cependant :slight_smile:

3 « J'aime »

une autre suggestion,
ce serait bien si vous pouviez ajouter un paramètre pour changer l’image de la bannière lumineuse, peut-être plusieurs options, ou changer aléatoirement.
disons, activer ce composant maintenant, il reste encore plusieurs semaines avant Noël, ce serait bien d’avoir quelques changements… :slight_smile:

1 « J'aime »

Pouvez-vous corriger les chapeaux pour mobile ? :grinning:

2 « J'aime »