laoyang/FamilyBook
1
0
Fork 0
Code Issues 29 Pull Requests Actions Packages Projects Releases Wiki Activity
147 Commits 1 Branch 0 Tags
306f5a61cbfa2cff69e13523b524964403fb4a66
Commit Graph
2 Commits
Author SHA1 Message Date
laoyang 8d79857f1a v54：首页任务标题截断（真根因=两列网格被撑破）/ 名册分组→标签且候选读词表 / 旅程地图视图 / 旅程设置按钮 / 显示方式默认列表或地图 
用户 2026-10-16 傍晚五条。

① 首页「近期任务」标题太长会挤成两行
   用户原话：「首页**近期任务**那个地方要注意截断：如果任务标题太长，
   会把这个任务**挤成两行**。」
   🔴🔴 **真根因是两层，第一版只修了表面**：
     · 表面：`.dash-what` 只有 `flex:1 1 auto; min-width:0`，**缺** nowrap/hidden/ellipsis
       ⇒ 收缩后文字在框内换行；
     · 真身：`.home-duo` 是**两列 grid**，grid 子项默认 `min-width: auto`
       （"我最小也得装下我的内容"）⇒ 一条长标题把那一列顶到 **1763px**
       （父 `home-duo` 才 1032px）⇒ **整个两列布局被撑破**、页面横向溢出到 **2089px**
       （视口 1440）。⇒ 行宽 1721px、`scrollWidth === clientWidth`，
       **`text-overflow: ellipsis` 看起来"没生效"** —— 其实容器自己先爆了。
   修法：`.home-duo > * { min-width: 0; }`，且 **整条 flex 链**都补
     （`.hd-pane` → `.hd-pane-head` → `.hd-pane-body` → `.dash-list/.tk-list` → `.tk-row`）
     —— 链上任何一层漏了，收缩就断在那一层、下面的 nowrap 全白写。
   配套：任务标题加 `title=` 悬停看全。
   ⚠️ 这是本项目**第二次**踩 flex/grid 的 `min-width:auto`（v53 的 `.mv-grid` 是同类）。
      **以后"文字截断不生效"先量容器有没有被撑破**。

② 名册「分组」→「标签」；打标签的候选要**从设置里读**
   用户原话：「现在名册里面给人打标签的时候，**这个标签数据是从哪读出来的**呀？
   在这个名册的设置页面，有很多分组，现在还叫分组。但是打标签的时候，
   **却不是从这些分组里读出来的**。……应该叫「名册标签」，是不是？
   **应该从这里面把所有标签都读出来**吧？……以后就没有分组的概念了，
   **全部都是标签的概念**。」
   🔴 **用户观察得完全正确，这是两套数据源**：
     · 设置页显示的词表来自 `person_groups`（**分组表**）；
     · 打标签的候选来自 `PersonTag::cloud()`（**你已经打过的词**，现算的）；
     ⇒ 两边根本不是一个数据源 ⇒ "设置里列着一堆、打标签时却只有几个"。
   修法：**`PersonTag::choices()` = 设置页词表 ∪ 我用过的词**（并集，词表在前）。
     · 词表里的词**还没用过**也能一点就打上（这正是词表的意义）；
     · 用过的词补在后面（历史不丢）；
     · 名册列表 / 新增 / 编辑 / 挑人 / 导入 **五处同一个函数**。
   界面口径全面改"标签"（**表名仍是 `person_groups`**：改名要背着 `PersonGroup` 类、
   `person_roster.group_id`、以及**另一个会话正在写的 `app/ApiHandler.php`**
   （它引用了 `PersonGroup::orderFor`）一起动，风险远大于收益）：
     · `_person_groups.php` 标题「名册标签」、按钮「加一个我自己的标签」；
     · 删除文案改成"只是从候选里去掉这个词，已打在别人身上的不受影响"
       （**语义真的变了**：标签是各自身上的字符串，不是引用）；
     · `people/_settings_menu.php`、`people/settings.php`、`settings/me.php`、
       `settings.php`（「公共分组」→「公共标签」）全部同步。
   顺带按"全部都是标签"改另两条入口：
     · `people/pick.php`「加进哪一组」下拉 → **打标签**；
     · `people/import.php` 同上；`ContactImporter::run()` 加 `$tags` 参数。
   🔴 新增 `PersonTag::addOne()`（**追加**一个标签）—— 这两处**绝不能用 `setFor()`**
      （它是"先 DELETE 再 INSERT"的整组替换）⇒ 会把人**原有**的标签清光
      （症状是"用一次导入，别人原有的标签全没了"，极难联想）。
   ⚠️ `group_id` 在 pick/import 的**服务端仍收着**（老书签/脚本兼容），只是界面不再展示。

③ 旅程模块加**地图视图**（读 TripLog 移植）
   用户原话：「你去读一下 TripLog 的代码，在旅程这个模块，**可以在地图上显示旅程数据**。
   1. 先读一下它之前那个代码是怎么实现的，**在我们的旅程模块也实现一下**。」
   读 TripLog（`templates/activities/index.php`）得到的要点：
     · 用**高德 AMap**（合规白名单内，绝不是境外瓦片）；
     · `position` 必须是 **[lng,lat] 数组**（传 `{lng,lat}` 裸对象会让 AMap 抛错、
       整个绘制循环当场中断 —— 它注释里写着"批 11 的光秃秃地图就是它"）；
     · `setFitView()` 后**上限 11 级**（只有一趟时会被放大到街道级，那是导航视角）；
     · `.amap-marker` 整层 `pointer-events:none`（否则包装层挡住别家卡片的点击）；
     · **容器必须有固定高度**（0 高度 → 空白地图）。
     它那套"同城环形扇开"是**几百行几何**，本仓数据量没压出这个问题 ⇒ **有意不做**。
   本仓数据更规整：`trip_nodes` 早有 `from_lng/from_lat/to_lng/to_lat`（v17 批）
   + `from_city/to_city` ⇒ 新增 `Trip::attachSpots()`：
     ① 有坐标取**重心**（不是端点 —— 端点会让"青岛→伊春"偏到某一头）；
     ② 没坐标退到**城市重心**（`Geo::lookup` 离线城市表，**不发网络请求**：
        列表页几十趟若逐条打高德逆地理，一次开页就是几十个外部请求）；
     ③ 都没有就不上图（列表里照常有，模板会说清"有几趟没上图"）。
     🔴 城市那里**必须也取重心**：`cities()` 的顺序是**按时间**（出发地在最前），
        取"第一个城市"会让**所有旅程都叠在家上**、地图毫无信息量。
   布局：两个视图用**标签页**（`?view=map` 是**链接**不是纯 JS 按钮 ——
     书签/前进后退/无 JS 都正确；也**有意不**把地图藏在 `display:none` 里，
     高德在 0 尺寸容器里会渲染空白）。
   高德 SDK 加载器重构：原来 `fbLoadAddr()` 顺带拉 `addr_suggest.js`，
   而地图只用 SDK 自带的 `AMap.Map/Marker` ⇒ 抽出 `window.fbAmapSdk()`（只拉 SDK）、
   新增 `fb-amap` meta（不带 plugin），`fb-addr` 那份共用同一加载器。

④ 旅程列表页「新建旅程」**后面**加「设置」按钮
   用户原话：「2. 在旅程的「新建旅程」后面也加一个「设置」按钮。」
   ⇒ 补 `_settings_menu.php`（设置恒排最右），菜单**第一条**加「显示方式」。

⑤ 显示方式：默认列表还是地图（用户自配）
   用户原话：「3. 里面的设置让用户自己配置：**到底默认显示旅程列表，还是用地图显示**。」
   ⇒ 新增 `trips/display`（设置页 + `?set=` 快速设置）与 `trips/display_save`，
     存 **`user_settings.trip_default_view`**（每人一份，**不是**全家设置）。
   ⚠️ 访客也能改（只影响自己看到的默认，不是数据变更）。

验证（新增 2 套件 + 更新 e2e/deploy 断言）：
 · _fb_v54_ui.py（**33 条**真机断言）
 · _b54_falsify.py（**11 注入 / 11 全抓**，基线 33/0）
 · e2e **1518/0**；familybook_deploy.sh 加「第54批形态」段（**24 条**）
 · 部署 **0 FAIL**；线上 **20 个文件 md5 逐字节一致**；
   线上实测：词表 24 条、`choices()` 24 条、**`cloud` 15 条全部 ⊆ `choices`**、
   4 趟旅程全部可上图（样例落在济南·青岛重心）；诊断页已删复验 404

⚠️ 本轮踩坑（已写进 pitfalls.md）：
 · 🔴🔴 **"文字截断不生效"要先量容器有没有被撑破**（第一嫌疑是 flex/grid 子项
   `min-width:auto`，不是 `text-overflow` 写错）；
 · **反向断言（"不该再出现 X"）必须用 `_code_has`（剥注释）** ——
   需求原话正当留在注释里，裸 `grep -F` 会命中自己的说明（本轮部署假红一次）；
 · **冗余防线会让反向验证打不红**（两处 `min-width:0` 只去掉一条 ⇒ 探针照旧全绿，
   注入必须一起去掉）；
 · **界面多加一个字符就可能打歪既有断言**（我在词表管理页给标签名加了 `#`，
   把 e2e 的 `my_group_order()` 读成 `#家人`）。
 2026-10-17 00:15:06 +08:00
laoyang ed122c034b v52：旅程 CalDAV 结束时间 / 笔记（标题·段落下加段加图·逐段就地编辑）/ 名册自动匹配与设置页 / 模块设置整合表 
用户 2026-10-15 七条。

① 旅程行程经 CalDAV 同步后「结束时间 = 开始时间」
   **真根因**（app/Calendar.php 的 exportRows()）：对**派生条目**（观影/旅程）写死
     'ends_at' => null，而 Ics::vevent() 遇到空结束会退化成「DTEND = DTSTART」
     ⇒ 手机上每条行程的结束时间都等于开始。
   **附带第二个 bug**：分组键含日期（kind-ref_id-date），而 month() 把**跨天条目逐天**
     铺出来 ⇒ 跨天节点被拆成 N 条（UID 相同），resources() 按 UID 去重后**只留第一天**，
     而第一天那条没有结束时刻 ⇒ 又回到 DTEND=DTSTART。
   修法：键改成 `kind-ref_id`，先收集同节点的各天，再用**首末天**拼一条真实跨度
     （末天那条才带 end_time）。实测单日 DTEND=12:30、跨天 DTEND=第3天11:00，
     每节点恰好 1 条 VEVENT。
   ⚠️ 副作用（已处理）：v31 DAV 去重的反向验证原来靠「跨天节点逐天出、UID 全同」造重复，
     本修把那条素材**修没了** ⇒ 给夹具换成「同一份 .ics 导两次」（两条 UID 相同的
     自定义日程），守卫仍能被证伪（见 _fb_dav_diag.py 与 pitfalls 的同族条目）。

② 笔记详情页标题是"看笔记"而不是笔记名
   阅读态 h1 改用笔记自己的 $noteTitle；**编辑态仍用「改笔记/写笔记」**
   （编辑时标题就在下面第一个输入框，两处重名看着像重复）。浏览器 <title> 同步。

③ 加段落/加图片要在**输入框下面**
   note_editor.js 的 makeBlock() 里每块下方加一排 .note-block-add（＋段落/＋图片），
   点了插在**这一块后面**；顶部那组**保留**（用户明确"上面和下面都保留"）。
   ⚠️ 挂在 body 里而不是 wrap 上 —— .note-block 是 display:flex 横排，
      挂 wrap 会变成"输入框右边的第三列"，不是"下面"。
   ⚠️ 用 data-add-below（不是 data-add）：顶部是"追加到末尾"、块内是"插在本块后"。

④ 笔记详情页「逐段就地编辑」（分块多时不用跳编辑页找那一块）
   模板给每块包 .note-read-item[data-block][data-kind][data-raw] + 右上角「编辑」按钮；
   新增 assets/js/note_inline.js（按需加载）：划过出现 → 点编辑就地变 textarea →
   保存 POST notes/block → 只换这一块。
   服务端 Note::updateBlock()：**按块主键改一行**（不是整组替换）——
   ⚠️ 整组替换会换掉每个块的 id（前端手里的 data-block 当场失效）+ 并发互相覆盖；
   ⚠️ SQL 必须带 `AND note_id = ?`（块 id 全局自增，少了它就能改别人笔记里的块）。
   权限与整条编辑同一把尺（find() + canWrite()）。

⑤ 删掉「从已有家里人中挑选」，改成「新增/导入时自动匹配」
   新增 Person::matchExisting()：手机号（有 phone_hash，一条 SQL 精确命中）→
   身份证（无哈希，整表解密比）→ 姓名（**先记下不立刻返回**，等精确命中看完再退到它）。
   people/create：校验通过后先 matchExisting，命中就在**同一张表单里**出确认条
   （#personConfirm）。⚠️ **不做成单独确认页**：单独页要把整张表（含工作经历/补充联系
   方式这些**数组字段**）原样回传，漏一类就静默丢数据；同表单里字段天然全在。
   ⚠️ 判"是不是第一次提交"用 array_key_exists('confirm_existing')（0=不是、1=是、无=首次）。
   导入那条路的缺口（本轮补）：ContactImporter 命中已有的人时只补字段、**不挂进导入者的
   名册** ⇒ "文件里有他、系统里也有他，但导完我的名册里看不到他"。已在命中分支加
   PersonRoster::ensure(..., $meId, ...)。
   顺手加 PersonRoster::has()（避免调用方又踩"visibleIds 是值列表不是集合"那个坑）。

⑥ 名册页只剩「添加成员 + 设置」；分组搬到名册设置
   templates/people/_settings_menu.php（新）：☰ 下拉两行 = 导入联系人 / 名册分组。
   templates/people/settings.php（新，路由 people/settings）：require 从个人设置**整块搬出来**
   的 templates/_person_groups.php。
   ⚠️ 是**真搬**不是复制（两份一模一样的分组界面，改一处忘一处是必然的）；
   ⚠️ 片段变量一律加 pg 前缀 —— 模板片段与宿主 require 是**共享作用域**，
      用 $groups 这种通名会覆盖宿主同名变量（本轮真踩到，已改）。
   ⚠️ 表单 action 仍指 settings/groups/*（一个字段没改）。

⑦ 个人设置：模块设置整合成一张表
   原来是**三张卡三个表单**（首页模块 / 导航模块 / 主题色），同一串模块名出现两遍。
   现在一张表五列：顺序 | 模块 | 导航 | 首页 | 颜色，一次保存。
   新增 settings/modules_save 路由：**底层存储一个没改**（仍是 nav_modules /
   home_modules / module_colors），只是把三个 save 依次调一遍 ⇒ 老数据天然兼容。
   ⚠️ 权限**没有**因为"合并"被放宽：颜色仍是家庭一份、只有管理员能改
      （非管理员那列是只读色块；提交时服务端静默忽略 c[]）。
   窄屏：表格 min-width:560px + 外层 overflow-x:auto（375px 实测横向溢出 0）。

验证（新增 5 套件 + 更新若干旧断言）：
 · _fb_v52_ics.py（3）/ _fb_v52_note.py（19）/ _fb_v52_roster.py（12）/ _fb_v52_mods.py（13）
 · _b52_falsify.py（**12 注入 / 31 断言全抓**）
 · _fb_import_test.py 补 ⑯（命中的人也要回到导入者名册）；_fb_dav_diag.py 换去重素材
 · e2e 1516/0；_fb_batch_e.py / _fb_e2e.py / familybook_deploy.sh 里随 #6/#7 搬家或改形的
   旧断言（分组在个人设置、模块行 li→tr、show_present、RRULE 前缀）意图一字不改
 · 全流水线 **PUSH_RC=0**、部署 0 FAIL（含"第52批形态"）；线上 16 文件 md5 逐字节一致

⚠️ 工程纪律（已写进 pitfalls.md / 工作日志）：
 · 探针自起的 PHP 服务**一律随机空闲端口**（写死 + "连上就复用" 会静默测到别的目录/库）；
 · **同一工具链目录 + 同一库名不支持两个会话并发** —— 本轮最后一个多小时的"诡异全红"
   全是**另一个会话**在同时跑 _fb_e2e.py、共享 fb_test 与 _fb_site 造成的；
 · 并发下**连 git stash 都不能用**（会把别人的半成品一起藏起来），基线对照用 git worktree；
 · 修好业务可能让"守着它的反向验证"空转（素材被修没了）⇒ 换一处等价素材，别删守卫；
 · 断言盯**形态**而非具体数值（分栏那次写死 50% 在别的批次数据上假红）。
 2026-10-15 23:04:25 +08:00