HelloGitHub-Team / geese

采用 Next.js 构建的开源爱好者社区。An open-source enthusiast community built with Next.js.
https://hellogithub.com
Apache License 2.0
296 stars 72 forks source link

fix: 调整了切换主题按钮样式 #122

Closed gjssss closed 1 year ago

gjssss commented 1 year ago

我发现切换主题按钮样式有一些问题

调整之前: image

调整之后: image

由于上面的按钮宽度是2.5rem padding是两个0.5rem border是两个1px,所以下面图标正确位移应该是2.5rem - 0.5rem * 2 - 1px * 2

521xueweihan commented 1 year ago
image

试了下,黑暗模式下边界不明显

gjssss commented 1 year ago

@521xueweihan 抱歉,我又观察了一下代码和页面,发现是因为这边还有一个left 1px image 导致还需要向右边移动一个像素,所以最后的结果应该是dark:translate-x-[calc(1.5rem-3px)] 抱歉搞错了,请问还需要再pr一下嘛

521xueweihan commented 1 year ago

@gjssss 感谢解决问题,不用啦,我来修改下就行了。

截屏2023-10-05 00 03 27
gjssss commented 1 year ago

@521xueweihan 好滴