Компонент рождественского декора 🎄

Настройка Компонент темы
25

Привет, @meghna, я отправил pull request с исправлением нескольких ошибок.

5 лайков
26

Спасибо @Canapin, PR принят. :+1:

4 лайка
27

Спасибо за этот отличный компонент темы.

Вы заметили, что декоративное изображение появляется слишком рано, когда включена боковая панель? Кажется, это происходит при ширине 1450 пикселей. При включённой боковой панели лучше подождать примерно до 1700 пикселей. Не знаю, возможно ли это.

2 лайка
28

Теперь я использую этот компонент вместо собственных кастомизаций, хотя мне больше нравятся круглые лампочки :blush:

Однако в компоненте всё ещё есть небольшая проблема: основания лампочек скрыты за заголовком:

основание лампочек скрыто за заголовком
основание лампочек скрыто за заголовком1247×639 146 KB

Я создал PR для исправления этого.

исправленные лампочки
исправленные лампочки1247×639 162 KB

Для тех, кто предпочитает круглые лампочки, можно добавить этот CSS в свою тему:

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

круглые лампочки
круглые лампочки1247×639 143 KB

5 лайков
29

Спасибо @Canapin, я принял PR. :heart:

5 лайков
30

Скриншот_20221221_143420_Chrome
Скриншот_20221221_143420_Chrome1075×277 83.6 KB

Нормально ли, что на смартфонах это так низко?

1 лайк
31

Отлично подмечено! Я посмотрю.

редактирование: Я создал PR.

3 лайка
32

Вертикальное положение будет неверным, если вы используете эту тему вместе с компонентом темы Welcome Banner, даже если баннер скрыт.

См.:

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

Самый простой способ обойти эту проблему — добавить следующий CSS в вашу тему:

.above-main-container-outlet.welcome-link-banner-connectors {
    margin-bottom: 0;
}
3 лайка
33

Я обновил компонент баннера со ссылкой на приветствие, чтобы избежать этого. Еще раз спасибо @Canapin!

3 лайка
34

Сегодня вышло обновление для этого компонента, поэтому я его обновил, даже если он здесь отключён, так как Рождество уже прошло. Теперь я вижу следующее:

Ошибка: недопустимый CSS после "...hadow: 0px math": ожидалось выражение (например, 1px, bold), а было ".div($globe-height," на строке 44 ../../../tmp/discourse_theme_acdcea7663ae5fb94fa2525e54768195/discourse-christmas-decorations/stylesheets/lights.scss из строки 2 common.scss >> box-shadow: 0px math.div($globe-height, 6) $globe-width*2 $globe-sprea --------------------------^

Что-то теперь сломано. :wink:

1 лайк
35

У вас установлена последняя версия Discourse?

1 лайк
36

У меня здесь запущена версия 3.1.0.beta2 [e64d1c4105]. Так что да, это бета-версия, но точно не последний коммит.

1 лайк
37

То же самое с [7eb6223b04] :wink:

1 лайк
38

Я пока откатил коммит и буду работать с @Falco над правильным исправлением.

1 лайк
40

Я тестировал это и обнаружил, что шляпы не отображаются в мобильном режиме.

41

@meghna
спасибо за эту прекрасную работу!

одно предложение: не стоит ли раскрасить колокольчики по обеим сторонам? сейчас они чёрно-белые, не очень празднично получается :smiley: но это уже твоё решение…

1 лайк
42

В мобильном виде списка участников нет. Если добавить шляпу для каждого автора темы, это будет выглядеть слишком перегруженно.
Кажется, сейчас на мобильном всё в порядке. Мое мнение.

43

Я выбрал чёрно-белый вариант, чтобы это хорошо смотрелось на любых цветовых схемах. Тем не менее, приветствуется pull request :slight_smile:

3 лайка
44

Ещё одно предложение: было бы здорово добавить настройку для смены изображения светового баннера — например, несколько вариантов или случайную смену. Например, если включить этот компонент сейчас, до Рождества ещё несколько недель, и было бы приятнее, если бы были какие-то изменения… :slight_smile:

1 лайк
45

Можешь починить шляпы для мобильных устройств? :grinning:

2 лайка