圣诞装饰组件 🎄

你好 @meghna,我创建了一个包含一些错误修复的拉取请求。

5 个赞

感谢 @Canapin,PR 已合并。 :+1:

4 个赞

感谢这个很棒的主题组件。

您是否注意到装饰图像在侧边栏打开时出现得太早?它似乎在 1450px 宽度时出现。侧边栏打开时,大约 1700px 可能会更好。我不知道这是否可能。

2 个赞

我现在使用此组件而不是我自己的自定义项,尽管我更喜欢圆形的灯泡 :blush:

但是,该组件仍然存在一个细微的问题。灯泡的底部隐藏在标题后面:

我已经提交了一个 PR 来修复它。


对于那些喜欢圆形灯泡的人,您可以将此 CSS 添加到您自己的主题中:

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

5 个赞

谢谢 @Canapin,我已合并 PR。 :heart:

5 个赞

使用智能手机的这个值这么低正常吗?

1 个赞

抓得好!我会看看的。


编辑:我提交了一个 PR。

3 个赞

如果将此主题与 欢迎横幅主题组件 一起使用,即使横幅已隐藏,垂直位置也会出错。

参见:

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 个赞

我已经更新了欢迎链接横幅组件以避免这种情况。再次感谢 @Canapin

3 个赞

今天此组件有更新,所以我更新了它,即使它在这里被禁用,因为圣诞节已经结束。现在我看到以下内容:

错误:在“...hadow: 0px math”之后出现无效的 CSS:在 common.scss 的第 2 行(来自 ../../../tmp/discourse_theme_acdcea7663ae5fb94fa2525e54768195/discourse-christmas-decorations/stylesheets/lights.scss 的第 44 行)预期为表达式(例如 1px、粗体),但得到的是“.div($globe-height,” >> box-shadow: 0px math.div($globe-height, 6) $globe-width*2 $globe-sprea --------------------------^

现在有些东西坏了。 :wink:

1 个赞

您使用的是最新版本的 Discourse 吗?

1 个赞

我在这里运行的是 3.1.0.beta2 [e64d1c4105]。所以,是 Beta 版本,但肯定不是最新的提交。

1 个赞

[7eb6223b04] 也是一样 :wink:

1 个赞

我现在已撤销提交,将与 @Falco 合作进行妥善修复。

1 个赞

我在测试这个功能时,发现在移动设备视图下看不到帽子。

@meghna
谢谢你这么棒的!

一个建议,两边的铃铛都应该上色吗?现在是黑白的,不太有节日气氛?哈哈,你决定就好……

1 个赞

在移动视图中,没有“贡献者列表”,如果为每个主题作者添加一个帽子,看起来会太忙了。
现在在移动设备上看起来还可以,我个人意见。

我选择黑白色是为了让它在所有类型的配色方案上看起来都不错。欢迎提交PR :slight_smile:

3 个赞

另一个建议,
如果能添加一个设置来更改灯光横幅图片,也许有几种选择,或者随机更改,那就更好了。
比如,现在就启用这个组件,离圣诞节还有几周时间,如果能有一些变化就更好了…… :slight_smile:

1 个赞

能为移动设备修复帽子吗? :grinning:

2 个赞