mirror of
https://github.com/Silent1566/webhtv.git
synced 2026-10-18 07:09:16 +00:00
docs(scrim): 侦察结论——弹窗 scrim 收敛的前提不成立,否决直接开工
审计 §6.3 提出「§9.7 弹窗 scrim 未走 token(15+ 处硬编码 dimAmount)」,我在上一批把它 记为「下一批首选」,理由是「刚接线的 overlayDark 正好可以收掉」。**这个判断是错的**, 本轮侦察推翻了它。本任务只交付结论,**不改任何生产代码**。 三条否决理由: 1) **dimAmount 不是颜色。** 规范 §9.7 原文是「scrim 使用 scrim」,而实测的 60+ 处 dimAmount 是**窗口级遮罩强度**(WindowManager.LayoutParams.dimAmount + FLAG_DIM_BEHIND), 由系统在窗口后面画一层黑,与任何颜色资源无关——它无法「改用某个 color token」。 真正对应规范那条的是**视图级遮罩**(布局里 background=@color/webhtv_color_scrim 的 全屏 FrameLayout)。 2) **webhtv_color_scrim 被混用。** 逐处核对 10 个用法,只有 3 处是真正的遮罩 (dialog_episode_detail / dialog_tmdb_person / dialog_tmdb_video_player 的根 FrameLayout), 其余 7 处是 1dp 分隔线(6 处)、88dp 行容器、海报占位底。全局替换会破坏这些非遮罩用途。 3 处真遮罩里还有 1 处(TmdbVideoPlayerDialog)保留 setDimAmount(0.72f),接线视图 scrim 会与窗口 dim **叠加**,不能套用另两个(它们都是 setDimAmount(0f))的做法。 3) **仓库已有一份专门设计文档给出过同一结论。** docs/THEME-SCRIM-OVERLAY-20260925-design.md 原文:「scrimOpacity 是实现遗漏,不是设计缺失。 但它当前被混用(分隔线/占位底),所以**不能对 webhtv_color_scrim 做全局替换**——必须按 视图语义限定作用域。」同文对 overlayOpacity 的裁定是「语义不成立,建议暂缓并降级为 Web 只读」。 **更重要的发现**:这是仓库**已登记的未完成项、且方案在候审**。 docs/THEME-CUSTOMIZATION-LAYERS-20260921.md 的状态行原文:「**当前剩余:scrimOpacity/ overlayOpacity 的接线(设计研究已完成、候审:docs/THEME-SCRIM-OVERLAY-20260925-design.md)** ……因此按第 8 节完成定义,**尚不能宣称『全局主题自定义已完成』**。」 即:它不是「审计新发现的缺口」,而是早就被识别、设计研究已完成、正等用户批准的工作项。 审计记这条时没有引用那份文档,我也误记为「下一批首选」——**正确做法是先读那份候审设计, 而不是自己另起一套**。 同时记录一条边界:我在 CONTRAST-P0-20261013 里把 webhtvColorOverlayDark 接进 Theme.WebHTV (为了让空态 scrim 取到值)。设计文档说 colorOverlayDark 是「4.23 已明确冻结(不随 profile 变)」。 接线本身不违反冻结(传的是固定色值,没做成用户可调槽),但 ThemeProfile.java:101 确实有 overlayOpacity 槽,两者一旦接上会互相牵动——这条边界要写下来。 需要用户裁定的两件事(本任务不代拟):① dimAmount 的 25 处非零值要不要收敛成语义档位 (0.58f 占 13 处像复制粘贴,但 0.18f/0.08f 是刻意的弱遮罩,不能一并统一); ② webhtv_color_scrim 的 7 处非遮罩用法要不要先改名(否则任何按 token 收口的改动都会误伤)。 环境限制:本会话无可用 bash,check_ui_tokens.sh 与 task_guard.sh 无法执行, 按同等意图手工暂存/提交/打 tag。
This commit is contained in:
parent
2cde387e55
commit
1055bd2b55
1 changed files with 99 additions and 0 deletions
Loading…
Add table
Add a link
Reference in a new issue