EveSunMaple / Frosti

A clean, elegant, and fast static blog template! 🚀 Developed with Astro
https://frosti.saroprock.com
GNU General Public License v3.0
181 stars 30 forks source link

Update BaseCard.astro #9

Closed YaoqxCN closed 3 months ago

YaoqxCN commented 3 months ago
vercel[bot] commented 3 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
frosti ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 1, 2024 0:36am
EveSunMaple commented 3 months ago

感谢您对此项目的支持,但是此 PR 存在一些问题:

  1. 我看到您添加了新的 badge ,辅助文字内容为“阅读量”,这些文字应该在 consts.ts 中有常量定义,而不是直接改在原文件中
  2. 事实上,几个 badge 还是贴在一起的,如果它们三个之间有适当的空隙,我想这会更好
  3. busuanzi 的数据统计存在一些问题,现在我在外面不能直接发给您,如果您用 lighthouse 审查应该可以发现问题所在——这也是为什么我删去了阅读量统计,也许有更好的解决方案
  4. 在统计量显示出来之前,元素内容为空,这不太合理

请确认上述问题并修改

再次感谢您的 PR!😊

YaoqxCN commented 3 months ago

检查了一下,接入 busuanzi 确实会影响网站(下图 lighthouse),已经把这个部分代码删除了。 image 至于文章顶部的 badge,我这里确实有间隔: image

EveSunMaple commented 3 months ago

你好像忘记改 EnvelopeCard.astro 的样式了……

无伤大雅,后面就交给我吧