Skip to content

优化多人联机反馈页面#294

Merged
zkitefly merged 5 commits intoHMCL-dev:mainfrom
neveler:dev-terracotta
Oct 18, 2025
Merged

优化多人联机反馈页面#294
zkitefly merged 5 commits intoHMCL-dev:mainfrom
neveler:dev-terracotta

Conversation

@neveler
Copy link
Contributor

@neveler neveler commented Oct 18, 2025

  • 直接跳转到登录页以减少匿名提交
  • 延长倒计时到 10 秒
  • 允许取消自动跳转
  • 隐藏目录
  • 移除了无实际内容的 help 与 token
  • 提取自动跳转逻辑为 jquery 插件
    • 插件会自动在页面首次加载完毕时获取所有 a[data-redirect] 元素并应用自动跳转,若元素为页面加载之后被插入 dom 则需要手动条用 jquery 方法如 $('#target').autoRedirect()

@zkitefly zkitefly merged commit 25324cd into HMCL-dev:main Oct 18, 2025
@neveler neveler deleted the dev-terracotta branch October 18, 2025 10:25
@burningtnt
Copy link
Member

burningtnt commented Oct 18, 2025

@zkitefly 能不能不要急着合并 PR…… 表单设计上就允许匿名提交,否则会严重降低填写的人数。该 PR 将被完全撤回。如果其他功能仍有必要合并,请重新提交 PR。

burningtnt added a commit that referenced this pull request Oct 18, 2025
@neveler
Copy link
Contributor Author

neveler commented Oct 18, 2025

@zkitefly 能不能不要急着合并 PR…… 表单设计上就允许匿名提交,否则会严重降低填写的人数。该 PR 将被完全撤回。如果其他功能仍有必要合并,请重新提交 PR。

  • 我注意到表单上的提示语“为避免刷票,烦请登陆后再填写哦~”,原本以为未登录的用户提交会被忽略,所以直接跳转到了登录页面。如果“匿名提交”是预期行为,建议将提示语修改为“为确保反馈数据可靠性,请优先考虑登录后提交,感谢您的反馈”。
  • 当前文档仓库缺乏活跃维护者,PR 创建后基本没有人提供反馈意见,这使得想参与贡献的开发者难以交流,导致 PR 的反复回滚。希望管理团队能优化 PR 的审核流程。
  • 其实你的需求无需回滚,只需将 href 属性改回原来的表单地址即可。

@zkitefly
Copy link
Member

抱歉,因为我自己看着没啥问题我就直接合并了,而且之前被叫不要设置 reviewer,而且没人来审核(

下次我会在审核完毕后隔个两三天再合并的

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.

3 participants