Skip to content

Fork/fix/refresh busy indicator pr 2#10

Merged
1186258278 merged 2 commits intoqingchencloud:mainfrom
youyli03:fork/fix/refresh-busy-indicator-pr-2
Mar 4, 2026
Merged

Fork/fix/refresh busy indicator pr 2#10
1186258278 merged 2 commits intoqingchencloud:mainfrom
youyli03:fork/fix/refresh-busy-indicator-pr-2

Conversation

@youyli03
Copy link
Contributor

@youyli03 youyli03 commented Mar 4, 2026

feat: 增加媒体目录放行开关

feat: 手机端发送消息后不会再弹输入法,便于阅览返回消息

  • 默认安全模式启动:npm start
  • 如需放开目录限制:MEDIA_ALLOW_ALL=1 npm start

youyli03 added 2 commits March 4, 2026 01:20
- 默认安全模式启动:npm start
- 如需放开目录限制:MEDIA_ALLOW_ALL=1 npm start
@1186258278 1186258278 merged commit 696ff9b into qingchencloud:main Mar 4, 2026
@1186258278
Copy link
Contributor

感谢 @youyli03 的第 5 个 PR!两个改动都很实用:

  1. 手机端键盘优化 — 双重检测(UA + pointer media query)方案很稳健,解决了移动端发消息后键盘遮挡回复的痛点,👍

  2. 媒体目录放行开关 — opt-in 设计合理。后续可以考虑改为目录白名单机制(如 MEDIA_ALLOWED_DIRS=/path1,/path2),比全开更安全。当前方案作为第一版完全 OK。

已合并 🎉 小建议:以后 PR 标题尽量和改动内容对应(这次标题沿用了上一个 PR 的名称),一个 PR 聚焦一个功能会更清晰。继续期待你的贡献!

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