|
|
|
@ -1318,12 +1318,136 @@ export function clampSocialBase(city: string, value: number): number { |
|
|
|
|
|
|
|
--- |
|
|
|
|
|
|
|
### 4.17 全站卡片圆角统一 8px + 去卡片阴影（2026-09-20 批次，接 4.16） |
|
|
|
|
|
|
|
#### 1. 起点：一条新增的硬规则，来自用户而非本轮排查 |
|
|
|
|
|
|
|
`AGENTS.md` 于 2026-09-20 14:12 新增一行（用户改动，本轮首次实现）： |
|
|
|
|
|
|
|
> 整个项目所有卡片的圆角都必须是8px，无阴影 |
|
|
|
|
|
|
|
同一次编辑还把提交规则从「每天 17：50 提交」改成「**提交并推送到远端**」。两处都写进本批。 |
|
|
|
|
|
|
|
**为什么不能靠 `grep 'borderRadius'` 一把梭**：项目从来没有「卡片」的样式键约定 —— |
|
|
|
同一屏里 `card` / `box` / `panel` / `section` / `wrapper` 混用，`cardBtn`（胶囊按钮）、 |
|
|
|
`cardTag`（标签）、`cardIcon`（圆形图标底）也共用 `card` 前缀。**先立判定口径，再动手**， |
|
|
|
否则「统一 8px」会连胶囊按钮和圆形头像一起压成方角，这是本批最大的一次性风险。 |
|
|
|
|
|
|
|
#### 2. 判定口径（本轮定义，后续沿用） |
|
|
|
|
|
|
|
| 判为「卡片」 | 判为「非卡片」 | |
|
|
|
|---|---| |
|
|
|
| 样式键为 `card` 或以 Card 结尾/内嵌（`scoreCard`、`emptyCard`、`preambleCard`） | 同名前缀的子件：`CardBtn`/`CardBullet`/`CardDot`/`CardBadge`/`CardIcon`/`CardWrap`/`CardTag`/`CardChip` | |
|
|
|
| 逐条读过块体、确认是「独立承载一组内容的面板」的 17 个非 card 命名样式（`emptyBox`/`emptyList`/`emptyHint`/`searchEmpty`/`pathSummary`/`section`/`cardBase`/`cardOffer`/`salaryBox`/`draftBanner`/`matchBox`/`aiTip`） | 按钮、标签、chip、输入框、Tab、列表行（`row`/`*Item`）、头像与圆点 | |
|
|
|
|
|
|
|
**圆 / 胶囊排除规则**：`半径 × 2 ≥ min(字面量宽, 字面量高)` 时判为圆或胶囊，**不动**。 |
|
|
|
命中并被正确跳过的实例：`cardBtn` 48 高 / 半径 24、`radioOuter` 18×18 / 半径 9、 |
|
|
|
`countdownCard` 60×60 / 半径 30、Holland 结果的类型序号圆标。 |
|
|
|
|
|
|
|
#### 3. 改动规模（`git diff` 实测，非估计） |
|
|
|
|
|
|
|
| 项 | 数量 | |
|
|
|
|---|---| |
|
|
|
| 圆角半径改为 8 | **147 处**（分布于 50 个 `.tsx`/`.ts`） | |
|
|
|
| 删除的阴影行（`shadowColor`/`shadowOffset`/`shadowOpacity`/`shadowRadius`/`elevation`） | **54 行**，7 个文件 | |
|
|
|
| 其中：删除模块级 `const shadow = {…}` 常量 + 其 `...shadow` 展开点 | `app/jobs/[id].tsx` 共 7 行 | |
|
|
|
| 卡片类样式总数 / 其中仍非 8px | **167 / 0** | |
|
|
|
|
|
|
|
改动前 `167` 个卡片类样式里，`130` 个由脚本改、`37` 个本来已是 8； |
|
|
|
另有 `17` 个走人工复核的面板命名，`130 + 17 = 147` 与 `git diff` 行数**逐一对上**。 |
|
|
|
|
|
|
|
#### 4. 两个审计脚本自身的缺陷（差点改错别人的圆角，如实记录） |
|
|
|
|
|
|
|
审计脚本（临时工具，不入库）第一版把**单行样式块**当成普通块解析： |
|
|
|
|
|
|
|
```ts |
|
|
|
modalCard: { backgroundColor: colors.white, borderRadius: 16, padding: 20 }, |
|
|
|
``` |
|
|
|
|
|
|
|
这类样式在本行内就闭合了，脚本却继续往下读，**把下一个样式块的值算成了它的值**。 |
|
|
|
后果不是漏改，是**可能取到邻居的半径然后改到邻居头上**。发现的路径是抽样复核时看到 |
|
|
|
`countdownCard` 被报成「半径 30 / 60×60 的圆」（它实际是单行 `borderRadius: 16`， |
|
|
|
那个 60×60 是隔壁 `countdownItem` 的），以及 `P3Identity` 的 `card` 被误判成 |
|
|
|
「100%×48 的胶囊」而整块漏掉 —— 修掉解析后它才被正确识别并改成 8。 |
|
|
|
|
|
|
|
第二个缺陷更隐蔽：`shadowOffset: { width: 0, height: 2 }` 里的 `width`/`height` |
|
|
|
被当成**元素尺寸**，于是所有带阴影的卡片都满足「半径×2 ≥ 尺寸」而被判为胶囊、 |
|
|
|
**整类静默跳过** —— 而带阴影的恰恰是最该改的卡片。修法是取尺寸前先剔掉 `shadowOffset` |
|
|
|
与全部 `shadow*` 行。修复后重新审计，`P3Identity` 等一批真卡片才浮出来。 |
|
|
|
|
|
|
|
两个缺陷都属于「**报错方式为静默漏判**」而非崩溃，所以本批不采信任何一次性的脚本输出， |
|
|
|
而是用 `verify-radius.js` 对 `git diff -U0` 的**每一行新增半径**反查它落在哪个样式块内 |
|
|
|
（结论：`147` 行中 `130` 行在 card 命名样式内、`17` 行在人工复核的面板白名单内、**0 行在名单外**）。 |
|
|
|
|
|
|
|
#### 5. 一次被拒绝的批量回滚，以及随之改走的「前向验证」 |
|
|
|
|
|
|
|
第一版脚本在 `app/onboarding.tsx:580` 崩溃（单行块无独立 `borderRadius` 行）， |
|
|
|
此时已写入 79 处改动。我尝试用 `git diff --name-only | grep -v AGENTS.md | xargs git checkout --` |
|
|
|
整体回滚重来，**被权限策略拒绝**（判定为对未提交工作的不可逆本地销毁）。 |
|
|
|
|
|
|
|
我没有绕过它，而是改走**前向验证**：不回滚，直接证明这 79 处（连后续的 68 处）全部合法。 |
|
|
|
事后看这个方向更对 —— 那次回滚还会连带**抹掉用户自己刚写的 `AGENTS.md` 编辑**， |
|
|
|
而当时我并不知道那份编辑的存在（它不在我这条命令的视野里）。**教训：把「回到干净状态」 |
|
|
|
当作修复手段之前，先确认干净状态里没有别人的东西。** |
|
|
|
|
|
|
|
#### 6. 刻意不动的两处（范围边界，不是遗漏） |
|
|
|
|
|
|
|
- `components/onboarding/ui/SliderCaptcha.tsx:294-297` 的滑块阴影**保留**： |
|
|
|
它是**控件**不是卡片，去阴影会让滑块与轨道在视觉上糊成一体。全库 `elevation:` / |
|
|
|
`shadowOpacity:` 现仅此一处（已 `grep` 全库确认）。 |
|
|
|
- `src/theme/colors.ts` 的 `shadow: '#000000'` token **保留**：`colors.test.ts:11` 仍在断言它。 |
|
|
|
代价是「卡片不再用阴影」这条约定**在 token 层看不出来**，属已知的文档-代码缝隙，未加注释（避免 |
|
|
|
为一个零引用的 token 增加维护面）。 |
|
|
|
|
|
|
|
#### 7. 覆盖边界：116 条「像卡片但不叫 card」的样式**未动** |
|
|
|
|
|
|
|
以「白/浅底 + 内边距或描边 + 圆角 ≠ 8 + 非圆非胶囊」反查，全库仍有 **116 条**这样的样式 |
|
|
|
（口径见审计脚本注释）。逐类看过：**几乎全是按钮、标签、chip、输入框、Tab 项、列表行、头像底** |
|
|
|
—— 它们不是卡片，压成 8 反而是错的。**这不是「差不多改完了」的余数，是本轮口径的边界**； |
|
|
|
若将来真要把某一类纳入，需要单独一次「这一类算不算卡片」的判定，不能顺手改。 |
|
|
|
|
|
|
|
#### 8. 真机实测（Android 模拟器 `emulator-5554`，截图核对，非代码走查） |
|
|
|
|
|
|
|
| 页面 | 看什么 | 结果 | |
|
|
|
|---|---|---| |
|
|
|
| `jobs` 岗位列表 | 卡片原本带阴影（`searchBox` 5 行） | 已平；8px 直角；仍能与 `pageBg #F6F8F9` 区分 | |
|
|
|
| `jobs/1` 岗位详情 | 3 张卡片 + 模块级 `shadow` 常量已删 | 已平；层级靠白底与描边区分，未塌成一片 | |
|
|
|
| `(tabs)/profile` 求职端 | `statsCard`/`enterpriseCard` | 8px；语义色（蓝/橙/绿）未受影响 | |
|
|
|
| `(enterprise)/enterprise-overview` | 4 张卡片共删 15 行阴影（本批最多） | 已平；数据块仍可辨认 | |
|
|
|
| `job-publish` | `card` 删 5 行阴影 | 已平；表单分区仍清晰 | |
|
|
|
| `tools/skill-cloud` | 4 张卡由 14 → 8 | 8px；**注：行业 Tab 行存在既有的纵向裁切/重叠，与本批无关，未追** | |
|
|
|
| `holland-result` | 渐变横幅 + 3 张类型卡 | 8px；内部序号圆标与标签胶囊**未误伤** | |
|
|
|
| `resume/export` | 预览卡 / 格式卡 / 模板卡 | 8px；模板缩略图内框圆角独立，未被压平 | |
|
|
|
|
|
|
|
**未实测**：`components/resume/EditKit.tsx`、`components/onboarding/ui/Modal.tsx`、 |
|
|
|
`components/onboarding/screens/P3Identity.tsx`、`LegalDocScreen` 的圆角改动只有代码级验证 |
|
|
|
（它们嵌在弹层/引导流/长文档里，深链直达需先走完前置流程），**未截图**。 |
|
|
|
|
|
|
|
#### 9. 本批次门槛复跑（2026-09-20 实测） |
|
|
|
|
|
|
|
`tsc --noEmit` **0 错误** · `eslint .` **0 error / 0 warning** · `npm test` |
|
|
|
**26 个文件 365 例全绿** · `check-routes` **0 失效跳转** · `check-affordances` **0 处** · |
|
|
|
`check:icons` **0 处无效图标名**（194 文件 / 1357 字形） · `check-backend-stubs` **140 个接口函数**。 |
|
|
|
|
|
|
|
#### 10. 本批次仍未做 |
|
|
|
|
|
|
|
- 第 7 条的 **116 条同名前缀样式**是否要纳入「卡片」口径（需产品/设计定夺，不擅自扩大）。 |
|
|
|
- 4.16 第 5/6 条的两项产品定夺（「良好」取色分歧、`'1年以内'` 死值）**状态不变**。 |
|
|
|
- 4.15/4.16 列的其余开放项**状态不变**（`companies` 全量 vs 抽样、`formatSalaryRange` 未下沉、 |
|
|
|
`chats: 8` 与测评勋章字面量、`CITIES` / `EXPECTED_CITY_OPTIONS` 两份词表有意保留、 |
|
|
|
数据层 `color: string` 未收紧）。 |
|
|
|
- **4.16 第七节列出的 A–E 五条重复声明发现**（skill-cloud 行业 Tab 6 选 3 有数据、 |
|
|
|
`GROUP_LABEL` ≡ `SKILL_GROUP_LABEL`、发岗表单选项表未上锁、「全部行业」同音异义、 |
|
|
|
jobs 页三个筛选 chip 落在假交互脚本盲区）**本批未实现**，已分析完，留待下一批。 |
|
|
|
|
|
|
|
--- |
|
|
|
|
|
|
|
## 五、后续增量开发建议（优先级） |
|
|
|
|
|
|
|
- **P0（上架前必做）**：~~图标替换~~ **应用图标【已完成，`878710c`，1024×1024「有职」logo】** · ~~隐私 / 协议内容填充~~ **隐私/协议草案【已存在，含 18 岁门槛，待填【待确认】占位】** · ~~EAS 初始化~~ **EAS 配置骨架【已完成，`owner` / `projectId` 两处占位符阻塞 `eas build`，需 `eas init` 注入，详见 4.5】** · ~~18 岁门槛核对~~ **18 岁门槛【已在隐私政策第六章 + 用户协议前言落地】** · ~~微信登录接入评估~~ **微信登录接入评估【已完成 → 见 `微信登录接入方案.md`，SDK 接入阻塞于微信开放平台 AppID】** · ~~启动屏接线~~ **启动屏【已完成，`expo-splash-screen` + `splash-icon.png`，需出包验证】**。 |
|
|
|
- **P0（补充 · 2026-09-15 新发现）**：① ~~页脚服务热线 / 举报监督电话待填~~ **已填 `15154608969`**，且 **`SupportFooter` 已挂在 P1Login / P2QuickLogin 两页上，页脚实际渲染**（2026-09-20 复核；4.10 第 7 条那句「无消费者 / 缺页脚组件」已撤回，见 4.14 第 7 条）；**隐私政策里【待确认】的客服电话应与之对齐**（同一个号，两处别填成两个）。② ~~缺「资质证照」页~~ **页面【已完成，见 4.10】—— 四层链路齐备（`src/types/qualification.ts` / `fetchQualifications` / `useQualifications` / `app/settings/qualifications.tsx`），设置页与「关于我们」页两处入口已恢复，三类证照的补图占位已逐张标明；**但证照的真实编号 / 发证机关 / 有效期 / 证照图仍全部待法务与资质方提供**（材料到位后只改 `MOCK_QUALIFICATIONS` 字段 + 更新测试断言），**未编造任何证件编号**。③ **注册引导链 + 简历导出的真机手工验证均未执行**（见 4.3 / 4.4 末条）。 |
|
|
|
- **P1（核心差异化）**：~~微信登录接入方案~~ **微信登录接入方案【已落文档，真实 SDK 接入阻塞于微信开放平台移动应用申请】** · ~~企业端快捷入口话术统一~~ **企业端快捷入口话术统一【已完成，6 处 label 对齐规范词；含沟通页快捷操作 onPress 接线，tsc/eslint 0 问题】** · ~~付费法律服务支付~~ **付费法律服务支付【已完成：mock 支付闭环，选套餐→结算→模拟支付→会员激活持久化，tsc/eslint 0 问题】** · ~~通关小游戏 7 关内容~~ **通关小游戏 7 关内容【已完成：结构原已建成，本次增厚至 7 关 22 题，tsc/eslint 0 问题】**。 |
|
|
|
- **P2（打磨）**：~~技术债清理~~ **技术债已清理（8 处 `any` → 明确类型，`eslint` 0 warning）** · ~~死控件清零~~ **死控件已清零（22 → 0，2026-09-17，见 4.8）** · ~~假交互清零~~ **假交互已清零（37 → 0，2026-09-18，**但 4.8 当时的「已完成」结论是错的，见 4.9 首条**）** · 真实后端对接 · OTA 通道验证。 |
|
|
|
- **P2（打磨）**：~~技术债清理~~ **技术债已清理（8 处 `any` → 明确类型，`eslint` 0 warning）** · ~~死控件清零~~ **死控件已清零（22 → 0，2026-09-17，见 4.8）** · ~~假交互清零~~ **假交互已清零（37 → 0，2026-09-18，**但 4.8 当时的「已完成」结论是错的，见 4.9 首条**）** · ~~卡片圆角统一 8px + 去阴影~~ **已完成（147 处圆角 + 54 行阴影，覆盖边界见 4.17；新增卡片须从 `src/theme` 出圆角常量这一层尚未建立）** · 真实后端对接 · OTA 通道验证。 |
|
|
|
- **P1（补充 · 2026-09-20 第三批，需产品定夺）**：① **「良好」这一档在个人页与其余五处取色不同** —— |
|
|
|
`app/(tabs)/profile.tsx` 的 `DIMENSION_LEVEL.good.text` 是 `textSecondary`（灰）， |
|
|
|
而岗位评级那五处一律 `primary`（蓝）。两处不同屏、后果有限，故**未擅自统一**； |
|
|
|
|
doublex001
1 month ago