Repository navigation
fix(client): raise mobile touch targets on coarse pointers - #520
Merged
Merged
Conversation
1 task done
Contributor
Author
|
Follow-up for the chrome side (collapsible header, collapsed notice summary) opened as #521 so this PR stays a touch-only fix. |
13 of 28 tasks
12 tasks done
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
摘要 / Summary
粗指针(手机)下,页面导航标签、头部控件与表单字段只有 28–37px 高。本 PR 在
src/client/MnemonView.module.css增加一段@media (pointer: coarse):把 masthead 控件、导航标签、分区操作、共享按钮变体和单行输入/下拉/文本域提高到 44px 触控基线,并给 canvas 加overscroll-behavior: contain以保住面板自己的滚动。细指针完全不受影响:所有规则都在媒体查询内,无标记与 API 变更。On a coarse pointer the navigation tabs, header controls and form fields are 28–37px tall. This adds one
@media (pointer: coarse)block: 44px hit areas for the masthead controls, nav tabs, section actions, shared button variants and single-line fields, plus scroll containment on the canvas. Fine pointers are untouched; no markup or API change.关联 Issue 或背景 / Related Issue or Context
N/A — CSS-only touch usability fix for existing surfaces; no new capability, Provider or persistence format.
涉及区域 / Affected Areas
PR 类型 / PR Type
最新代码确认 / Latest Codebase Confirmation
main分支开发,或在提交前已 rebase 或合并最新main。 / I developed from the latestmain, or rebased or merged the latestmainbefore submitting.同步命令 / Sync command:
git fetch origin && git rebase origin/mainAI 编码披露 / AI Coding Disclosure
使用的 AI 模型 / AI model used:
deepseek-v4.1-flash使用的编码 Agent 工具 / Coding Agent tool used: DeepSeek Harness (
dsh) coding agent仓库规范检查 / Repository Rules
@deepseek-ai/*NPM 契约。src/host/protocol.ts为准,没有在两侧重复定义 wire DTO。lib/文件。.changeset/mobile-touch-targets.md,dsh-mnemon: patch)。兼容性与数据安全 / Compatibility and Data Safety
N/A — CSS only. 不影响 DSH/Mnemon 版本兼容、Provider、配置、存储格式、升级或回滚,也不影响现有数据。
本地验证 / Local Validation
执行的命令 / Commands run:
结果摘要 / Result summary:
verify:docs通过。test:root:158 个文件通过、2 个跳过;2257 个用例通过、7 个跳过。构建期需要各 workspace 插件的lib/,因此先逐个构建了plugins/*(与verify的顺序一致)。tests/fixtures/presentation-baseline.json已更新:该指纹会统计规则条数,本次新增 10 条(1059 → 1069);class map 未变(289 个类,哈希不变)。pnpm run verify(Headless Profile 激活与 package lint 两个阶段);剩余风险限于这两项,由 CI 覆盖。用户可见变更证据 / Local Feature Evidence
证据 / Evidence:
用构建产物
lib/client.js的样式表在 390×844、粗指针下渲染:夹具使用构建后的类名拼出的最小 DOM,不含凭据、私有记忆或敏感路径;这是样式表尺寸测量,不是真机验收。