Conversation
|
深色模式有很多细节问题,例如tag鼠标悬浮后看不清文字,tag汇总页面顶部tag看不清,目录背景不再和文章背景保持一致,深色背景下横线都是白的很刺眼,tag汇总页面标题hover后颜色不再变化,搜索栏已经输入的文字看不清,等等 |
|
已收到反馈。 |
|
新加的评论模块是以深色模式显示的,不知是否会被一并修复? |
giscus的主题配置在_config.yml下,主题设置为: 会考虑修改成适配主题转换的模式。 |
- 新增深色模式样式文件 (less/dark-mode.less, css/dark-mode.css) - 在导航栏添加主题切换按钮 (月亮/太阳图标) - 使用 localStorage 保存用户主题偏好 - 实现无闪烁加载机制 - 完整适配所有页面元素的深色主题 - 更新 Gruntfile.js 支持编译深色模式样式 - 优化夜间阅读体验 - 修复了多个 UI 细节问题 - 添加 !important 确保深色模式样式优先级
- 在 _config.yml 中添加 theme 配置选项(light, dark, auto) - 当 theme 设置为 'auto' 时,Giscus 自动跟随网站深色模式 - 使用 MutationObserver 监听 data-theme 属性变化 - 当用户切换深色模式时,自动更新 Giscus 主题 - 支持其他主题配置(light, dark)保持不变
b82fa49 to
0ad4f54
Compare
|
已修复 |
| } | ||
|
|
||
| // Scrollbar (Webkit) | ||
| ::-webkit-scrollbar { |
There was a problem hiding this comment.
亮色就没有调 scrollbar 啊,不需要调 scrollbar,而且 -webkit- 在 Firefox 不生效
|
下次不要 force push,不然看不出修改 less 到底改了哪里 |
但可能会导致commit太多 |
|
我先修改吧,具体哪里显示有问题可以截图我在修改 |
|
好的谢谢 |
- 在选择器(h1, h4, article, blockquote 等)周围添加一致的间距 - 标准化注释的格式和位置 - 在逻辑规则组之间添加空行,以提高可读性 - 使用正确的换行来规范选择器分组(.table th, .table td) - 采用一致的格式对齐供应商前缀和过渡效果 - 在不影响功能的前提下,提高整个样式表的代码一致性
- 移除深色模式中的 scrollbar 自定义样式 - 保持滚动条样式在亮色和深色模式下一致
- 添加 .post-title hover 时变色效果 - 与亮色模式保持一致 - 添加 text-decoration: none 保持样式一致
- 移除 side-catalog 的 box-shadow - 目录背景与页面背景保持一致 - 消除上面的横线效果
- 根据主题动态设置标签云颜色范围 - 浅色模式: #bbbbee 到 #0085a1 (浅紫到深蓝) - 深色模式: #4a9eff 到 #b8d4ff (浅蓝到中深蓝) - 主题切换时自动重新初始化标签云颜色
- 移除 mixins.less 中 font-family 内的注释(LESS 不支持) - 移除 Markdown 链接语法(导致编译错误) - 现在 hux-blog.less 能正常编译
- 修改 index.html,使用 .disabled 类替代内联样式来实现禁用的翻页按钮,使 HTML 标记更清晰。 - 在浅色和深色模式中都添加了 pointer-events: none,确保禁用的按钮无法被点击。 - 为禁用状态添加了专用的 LESS 变量 (@disabled-bg, @dark-disabled-bg, @dark-disabled-text)。 - 更新了深色模式下禁用按钮的颜色,以获得更好的对比度。 - 为深色模式的禁用按钮样式添加了 !important 标志,确保其优先级和光标样式 (cursor: not-allowed) 生效。 - 移除了分页标记中不必要的内联 pointer-events 样式。
|
回复一下,首先我确实使用AI进行vibe coding,但是功能落成也确实需要人来code review,新增的功能本来就可能存在问题,之前也没详细一个一个去改。提出意见后修复是很正常的事情,但是不能用模糊的描述让我去修改,我怎么知道你的标准? |
抱歉,不应该假设你调试 css 的经验较多(这些问题,尤其文字看不清的问题,如果有经验应该很容易识别或修复,因此最初只用文字描述)。仍然感谢你的贡献。欢迎继续贡献! |
感谢你的评审,我主要是写后端的,前端的细节确实有很大问题,您的图片帮我解决了定位的问题,这次都是手动修改,消息使用AI辅助生成,再次表示感谢,我也有很多收获。 |
1 similar comment
感谢你的评审,我主要是写后端的,前端的细节确实有很大问题,您的图片帮我解决了定位的问题,这次都是手动修改,消息使用AI辅助生成,再次表示感谢,我也有很多收获。 |
|
如有问题请反馈,但是请不要带有歧视性的话语,谢谢 |
真的很抱歉🙏 |
|
已收到反馈,我会尽快修改,有歧义的地方我会给方案。谢谢您的反馈意见 |
- 修改 footer.html 中 tagcloud 初始化,添加 100ms 延迟确保 CSS 变量已应用 - 添加 fallback 颜色,CSS 变量读取失败时使用默认值 - 更新 nav.html 主题切换代码,使用 CSS 变量而非硬编码颜色 - 添加 fallback 确保主题切换时颜色正确显示 - 修复亮色模式下 tagcloud 显示灰色的问题 - 确保手动切换暗色模式时 tagcloud 颜色正确更新
- 在 colors.less 中添加 @light-catalog-active-bg 和 @dark-catalog-active-bg 变量 - 在 variables.less 中添加 @catalog-active-bg 默认值 - 在 dark-mode.less 中添加 @catalog-active-bg 暗色模式覆盖 - 更新 side-catalog.less 使用 @catalog-active-bg 变量替代硬编码 #F5F5F5 - 确保 side-catalog 在亮色和暗色模式下都使用正确的配色
- 为深色模式下的文章副标题(post-subtitle)添加悬停(hover)和焦点(focus)状态的颜色样式(#a78bfa)
- 将 transition 属性从 .post-preview > a 移至全局 a 选择器 - 添加完整的浏览器前缀支持 (-moz, -webkit, -o) - 确保所有深色模式链接具有一致的过渡效果
|
7.或者使用另一种方案,给nav加上透明属性 |
- 为 .post-heading .tags, .page-heading .tags, .site-heading .tags 添加透明背景样式 - 使用 rgba(255, 255, 255, 0.3) 的透明边框 - 使用 rgba(255, 255, 255, 0.9) 的透明文字颜色 - hover 时使用 rgba(255, 255, 255, 0.2) 的透明背景 - 保持与亮色模式的一致性,提高可见性
如果没有拆分修改到不同提交的需求,仓库维护者在合并 PR 时可以选择 "squash and merge", 这样就只有一个提交了。 |
感谢解答。 |
|
@h1s97x : 这个 14c415e 提交就是挤压、压缩(squash)到一起的。文档: Squash and merge your commits § About pull request merges - GitHub Docs. |























功能描述
为京吹学报添加完整的深色模式支持,提升夜间阅读体验。
主要特性
✨ 深色模式样式
less/dark-mode.less,css/dark-mode.css)🌙 主题切换
⚡ 优化体验
修改文件
Gruntfile.js- 更新构建配置以支持编译深色模式样式_includes/head.html- 添加主题加载脚本(防止闪烁)_includes/nav.html- 添加主题切换按钮css/dark-mode.css- 编译后的深色模式样式less/dark-mode.less- 深色模式 LESS 源文件less/hux-blog.less- 更新主样式文件测试方法
浏览器兼容性
相关 Issue
无
备注