Skip to content

Change MOUSE_FOLLOW default to true#4093

Open
townesholer wants to merge 1 commit into
notionnext-org:mainfrom
townesholer:patch-2
Open

Change MOUSE_FOLLOW default to true#4093
townesholer wants to merge 1 commit into
notionnext-org:mainfrom
townesholer:patch-2

Conversation

@townesholer
Copy link
Copy Markdown

尽量按此模板PR内容,或粘贴相关的ISSUE链接。

已知问题

  1. (示例)版本号管理不规范
    • 版本号直接写在环境变量中,容易出错
    • 多处维护版本号,可能不一致

解决方案

  1. (示例)将版本号管理从 .env.local 迁移到 package.json
    • 统一从 package.json 读取版本号
    • 使用 IIFE 优雅处理版本号获取逻辑
    • 保持向后兼容,支持环境变量覆盖

改动收益

  1. (示例)更规范的版本管理
    • 统一从 package.json 读取
    • 保持与 npm 生态一致
    • 减少人为错误

具体改动

  1. (示例)blog.config.js
    • 移除原有的静态版本号配置
    • 在文件末尾添加动态版本号获取逻辑
    • 保持向后兼容,优先使用环境变量
    • 添加错误处理和默认值

测试确认

  • 本地开发环境测试通过
  • 生产环境构建测试通过
  • (如适用)版本号正确显示
  • (如适用)环境变量配置正常工作

用户文档(docs/user-guide/

若本 PR 修改 docs/user-guide/docs/developer/ 中与站长相关的说明,可勾选「不适用」并跳过本节。

  • 不适用(无文档改动)
  • 已按 维护工作流 自检
  • 路径符合 docs/user-guide/ 目录约定
  • 已更新 user-guide/README.md(新增/移动文章时)
  • 已更新 ARTICLE_INDEX.md(新 slug 或路径变更时)
  • 环境变量名与 conf/*.config.js 一致(若文档涉及配置)
  • 示例中无真实 Token、.env、私有 ID
  • 保留或更新了「原文链接」(若源自 docs.tangly1024.com)

文档说明(可选):对应官方 slug / URL、是否与功能 PR 配套

Copy link
Copy Markdown
Collaborator

@tangly1024 tangly1024 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

当前改动不能直接合入:它会让所有未显式配置的站点默认加载 /js/mouse-follow.js 并运行 canvas 粒子动效;移动端只是 CSS 隐藏 canvas,脚本仍会加载和初始化。这属于全局默认行为、性能和可访问性变化,但 PR 未提供需求来源、文档更新或构建/交互验证。请保持默认关闭,或补充明确的产品决策与按设备/降低动态效果偏好禁用的实现后再评估。另建议后续单独处理环境变量字符串 alse 仍会被视为开启的问题。

@tangly1024
Copy link
Copy Markdown
Collaborator

补充更正:上条 review 最后一项所指的字面配置是 NEXT_PUBLIC_MOUSE_FOLLOW=false;当前 process.env.NEXT_PUBLIC_MOUSE_FOLLOW || false 会把非空字符串 false 当作开启值。该行为是独立问题,不建议通过将默认值改为 true 解决。

@tangly1024
Copy link
Copy Markdown
Collaborator

再次更正:我继续检查了 lib/config.jssiteConfig() / convertVal() 路径,确认消费配置时会把字符串 false 解析为布尔值 false,因此环境变量关闭行为不构成本 PR 的问题。request changes 的依据保持不变:本改动会默认启用全站 mouse-follow canvas 动效,但目前缺少产品决策、性能/无障碍处理及验证。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants