laoyang/FamilyBook
1
0
Fork 0
Code Issues 29 Pull Requests Actions Packages Projects Releases Wiki Activity

v24 第 24 批：选人冲掉已有成员 / 首页改版 / 个人导航 / 微信读书 / 财务图

Browse Source 
用户 6 条原话（要点）：
  ①「选择名册里面人员的那个弹出框有个问题，就是会把页面里面已有的人员给冲掉。
     比方说电影里面的观影人或者是旅程里面的参与人…这是一个 bug 应该通篇检查。」
    并按「输入拼音 → 空格选中第一个 → 输入框清空回焦 → 再输下一个 → 回车=确定」重做交互。
  ② 首页简要信息改成：问候 + 姓名 + 日期 + 星期 + 农历 + 节气/节日/纪念日/生日，
     下面「今天几条日程 / 几件待办」，天气**彩色图标**放**最右侧**，温度风力，最下面地址。
  ③「家里的镜框这里呃全部不要了」→ 删模块卡片墙 + 未来七天；日程与任务**左右各半**、
     高度固定、内部滚动且**不显滚动条**；日程**掐头去尾** + 当前时刻**红线**；
     任务显示标题+状态+**所在看板名**；最下面往年的今天。
  ④「把个人导航项目完整搬过来，放在读书后面」；免登录展示页 `/nav/?<短语>`；
     右上角给**登录**入口；展示页头像用**用户自己的头像**。
  ⑤ 读书加笔记 + 导入微信读书笔记（用户明确「**用微信读书 api key**」）。
  ⑥ 财务页第二张多线图不随类别标签筛选。

用户拍板（AskUserQuestion）：① 首页删「模块卡片墙 + 未来七天」两块；
② 微信读书用官方 API Key；③ 导航短语「自己起名 + 可一键换随机码」都做；
④ 6 条一次做完再上线。

════ 一、 🔴 第 1 条：复现出的真 bug（全站性）════
**根因**：选人清单是**打开弹框时才建 DOM**（懒建，为性能），而承载"已选"的
checkbox 就在那份清单里 → **没打开过弹框就提交**时表单里一个值都没有 →
后端把"表单里没有 attendees"理解成"用户要把人全删掉" → **静默清空**。
实测复现：库里同行人 `2,3,4` → 只改标题保存后变成 **NULL**。

修法：模板加**承运字段**（`.pp-carry` 隐藏 input，承载当前已选）；
JS 建清单时**撤掉**它（否则同一个 name 会带两份值）。
⚠️ 裸 POST（完全不带 attendees）**本来就该清空** —— 那是显式地"不选任何人"，
所以断言也必须照着"浏览器实际提交的样子"（带承运字段）来测。

**键盘流程**（用户第 1 条后半）：空格 = 选**当前可见列表里第一个还没选的**
（⚠️ 必须跳过已选中的 —— 否则会把它**取消**，与"我要选人"正好相反，本轮真机跑出来的缺陷）；
选中后清空输入框 + 光标回输入框；回车 = 确定。空格必须 `preventDefault`
（否则输入框里多一个空格字符）。

覆盖范围：4 个多选（`attendees[]` / `members[]` / `shares[]` / 修订 `members[]`）
+ 全部单选（归属人 / 负责人 / 纪念日人等）。

════ 二、 首页改版（第 2、3 条）════
自上而下只有三块：
  ① 信息面板：问候（早/中/晚 + 姓名）→ 日期 / 星期 / 农历 → 日子标记
     （法定节假日 / 节气 / 传统节日 / 纪念日 / 生日，各一色）
     → 今日数字（今天几条日程、几件待办）｜ **最右**：彩色天气图标 + 温度 + 风力 + 地址
  ② 日程 / 任务 **左右各半**（各 360px 固定高、内部滚动、**不显滚动条**三套写法）
     日程是**日视图形式的刻度**、**掐头去尾**（首末日程前后各 1h）、**当前时刻红线**
     任务是标题 + **状态** + **看板名** + 截止
  ③ 往年的今天

新增 `Home::todayMarks()`（把五类来源合成一句话："今天是国庆节 · 大暑"）与
天气的 `tone` 色调档（彩色图标靠 CSS 上色，**不用 emoji** —— 避免旧系统豆腐块）。
删掉：`mod-cards` 模块卡片墙、`未来七天`。

════ 三、 个人导航（第 4 条，SimpleNav 完整搬运）════
`app/Nav.php`（4 张 `nav_*` 表 + 稀疏排序 + 短语）· `app/NavFetch.php`
（抓取与图标生成，27KB，从原项目 functions.php 机械搬运，**原文那批"线上实测"注释全部保留**）
· `nav_show.php`（**全站唯一免登录页**）· `nav_suggest.php`（联想代理，凭短语门禁）
· `templates/nav/index.php`（管理页：板块/站点/引擎/设置 + 拖拽排序）。

搬运只改两处（不改会坏）：① 独立 SQLite → 家书的 MySQL；② **所有查询加 `person_id`**
（原项目是单用户系统，不加这一维全家人会共用同一份导航）。

免登录展示页：`/nav/?<短语>`（也认 `/nav/?<短语>` 不带参数名）；
短语用户自起名（4~32 位字母数字`_-`，全局唯一、保留词拦截）或一键换随机码；
右上角**登录**入口（进家书登录页）；头像用**他自己**的头像；空态给一句话。

路由三处同步（`router.php` / `.htaccess` / 部署脚本 5c **独立标记块**）——
这是本仓库第三次复用这条规律：**每加一个"站根上的独立入口"，就必须是自己的标记块**。
⚠️ 线上重写规则**顺序有语义**：`^nav/suggest\.php$` 必须在 `^nav/?$` 之前
（后者会把前者吞掉）。

════ 四、 微信读书（第 5 条）════
用户提示「用微信读书 api key」是对的 —— **我的知识过时了**：2026 年官方已开放
（网关 `POST https://i.weread.qq.com/api/agent/gateway`、`Bearer wrk-` 认证、
接口 `/user/notebooks`（游标 lastSort 分页）、`/book/bookmarklist`、
`/review/list/mine`、`/book/chapterinfo`）。

`app/WeRead.php`：Key 用 `Crypto::encrypt` **加密落库**、界面只回显掩码
（官方原话：这串 Key 等价于账号凭据）；导入走「拉预览 → 勾选 → 导入」，
落 `book_notes`（去重 = kind + 内容 + 位置）。
⚠️ 业务参数必须**平铺在顶层**，不能包进 `params`（官方文档专门列了这个反例）。
另一条路「手动粘贴」保留（`books/notes_import`，早已有）。

════ 五、 财务第二张图（第 6 条，**上一批我引入的 bug**）════
`g.hidden = true` 对 **SVG 的 `<g>` 无效**（它没有 `hidden` 属性）→ 线条一条都没被藏；
**而我的探针也读同一个属性** → 两边一起绿（假绿）。
改为 `style.display`（恢复用 `''` 而不是 `'block'`）；探针改读
`getComputedStyle(el).display !== 'none'`（落在**渲染结果**上）。
⚠️ "探针与实现读同一个属性"是**互相掩盖**的典型，比单纯写错断言更危险。

════ 六、 验证 ════
e2e **1218/0**（本批新增 102 条）· 真机探针 **19/0**（选人）+ **24/0**（首页/导航）
+ **34/0**（导航模块）· 桌面几何 0 失败 · 移动端几何 674/0 · 老库升级 149/0
· 部署静态断言空跑 **310/0** · **反向验证 13/13**（逐条改回旧行为，点名断言都变红）
· 线上验收：9 个新文件全在、4 张 nav 表建好（schema_version=24）、
  三条重写规则顺序正确、免登录页 / 空短语 / 错短语 / 联想门禁 全部响应正确。

════ 七、 本批的"夹具与断言"教训（都写进注释了）════
· `calendar_events.all_day` **默认值是 1** → 插夹具不写它就成了全天事件 →
  `time` 被抹成 `''` → "刻度掐头去尾"看着像没做（白查三轮）。
· 日历本要挑 `kind='custom'`（我原以为"普通本没有 kind"，实际**普通本就是 custom**）。
· 量"同高/同排"要拿**内容区**（去 padding）比，不能拿 border-box（`.topline` 有 14px 上内边距）。
· 判据不能把**边界上的正确行为**当 bug（22 点有日程时末刻度**就是** 23:00）。
· 「HTML 写了、CSS 没写」e2e 永远查不出来 —— 导航展示页的登录按钮就落到了左边
  （实测按钮右 116px / 整行右 1210px），只有真机量几何才抓得到。
· 反向验证里 e2e 会**重铺副本**把注入覆盖掉 → 给 e2e 加了 `FB_PATCH_FILE` 钩子
  （重铺完自动打补丁），13 例里 8 例"没牙齿"其实是这个假象。

════ 八、 新增/改动文件 ════
新增：`app/Nav.php` · `app/NavFetch.php` · `app/WeRead.php` · `nav_show.php` ·
`nav_suggest.php` · `templates/nav/index.php` · `templates/books/weread.php` ·
`assets/css/nav-show.css` · `assets/js/nav-show.js`
改动：`app/Schema.php`（v24 + 4 张 nav 表）· `app/Module.php`（+nav）· `app/Home.php` ·
`index.php` · `templates/_person_picker.php` · `assets/js/person_picker.js` ·
`templates/home.php` · `templates/ledger/index.php` · `templates/books/_settings_menu.php` ·
`assets/css/app.css` · `router.php` · `.htaccess`
This commit is contained in:
laoyang
2026-10-07 05:52:40 +08:00
parent ae3060f552
commit 09e30d27d4
21 changed files with 5034 additions and 180 deletions
Download Patch File Download Diff File Expand all files Collapse all files
assets/js/person_picker.js
+83 -3
View File
@@ -372,6 +372,14 @@
var type = list.getAttribute('data-pp-type') || 'radio';
var fname = list.getAttribute('data-pp-name') || '';
// 🔴🔴 先把**承运字段**撤掉（v24 第 24 批 用户第 1 条修的真 bug）。
// 它是服务端渲染的隐藏 input，承载"当前已选"；一旦真的清单建出来，
// 真值的输入项就在 DOM 里了，承运字段必须**立刻移除** —— 否则提交时
// 同一个 name 会带上两份值（承运一份 + 真值一份），后端收到重复项。
// 详见 templates/_person_picker.php 里那段长注释。
Array.prototype.forEach.call(
root.querySelectorAll('.pp-carry'), function (el) { el.parentNode.removeChild(el); });
var frag = document.createDocumentFragment();
people.forEach(function (p) {
var lab = document.createElement('label');
@@ -555,9 +563,81 @@
document.addEventListener('keydown', function (e) {
if (e.key === 'Escape' && !modal.hidden) { closeModal(false); }
// 多选弹窗里按回车 = 点确定（用户填完最后一个字母顺手回车很自然）
if (e.key === 'Enter' && !modal.hidden && stagedMode
&& e.target === search) {
if (modal.hidden || e.target !== search) { return; }
/* ============================================================
键盘流畅流程（v24 第 24 批 用户第 1 条后半，用户原话）：
「当我输入了一个人的拼音、首字母的几个字母之后，如果第一个列表里面，
第一个人就是我要选的这个人。这时候我按空格就直接把这个人选中…
只要是完成了选中一个人，这个操作之后，就把输入框里面的用户已经输入的
字母全部给它清空。同时光标回到输入框里面，让用户等待的选出下一个人的
字母。…这时候如果它回车，就是确定，就是和点击那个框上的确定按钮
是一个效果。」
⇒ 所以这里要三件事（按用户说的顺序）：
① 空格 = 选中**当前筛选结果里的第一个人**；
② 选中之后：清空输入框 + 光标回到输入框（接着打下一个人的拼音）；
③ 回车 = 等于点「确定」（把已选的人写回页面）。
⚠️ 只在**输入框里**按才生效（`e.target !== search` 直接返回）：
否则用户在弹框其它地方按空格（比如点完按钮后焦点在按钮上）会误选人。
⚠️ 空格**必须** preventDefault：`<input type="text">` 里空格默认会打出一个空格
字符，不挡的话输入框会变成 "yyq "（多一个空格，看着像拼音打错了）。
⚠️ 「第一个人」取的是**当前可见**列表的第一个（applyFilter 已经算好 `hidden`），
而不是 DOM 里的第一个 —— 用户看到的顺序才是他心里的"第一个"。
这也是用户那句"如果第一个列表里面，第一个人就是我要选的这个人"的意思。
============================================================ */
if (e.key === ' ' || e.key === 'Spacebar') {
e.preventDefault();
// 清单可能还没建（懒建）—— 空格时先补建，否则一个候选都找不到
buildList();
// 🔴 「第一个人」要取**当前可见、且还没被选中的**那一个。
//
// 为什么（这是本轮真机跑出来的一处交互缺陷）：
// 用户描述的是「输拼音 → 按空格 → 就选中这个人」。而如果**第一个可见的**恰好
// **已经选过**（比如他没清筛选、接着按空格），`!checked` 会把它**取消掉** ——
// 与他"我要选人"的意图正好相反，而且他多半看不出来（人少了以为是没选上）。
// ⇒ 跳过已选中的，选"第一个还没选的"；一个都没得选时**什么都不做**
// （不要静默取消任何人）。
var first = null;
var items = list.querySelectorAll('.pp-item');
for (var i = 0; i < items.length; i++) {
if (items[i].hidden) { continue; }
var _b = items[i].querySelector('input');
if (!_b || _b.value === '0') { continue; }
if (!_b.checked) { first = items[i]; break; } // 只认"还没选的"
}
if (!first) {
// 可见的都选过了（或一个候选都没有）→ 什么都不做，
// 只把"没有匹配的人"的提示按需亮一下，**绝不取消任何已选**
var _any = false;
for (var j = 0; j < items.length; j++) { if (!items[j].hidden) { _any = true; break; } }
if (!_any && emptyTip) { emptyTip.hidden = false; }
return;
}
var fbox = first.querySelector('input');
if (!fbox || fbox.value === '0') { return; }
if (stagedMode) {
fbox.checked = true; // 见上：到这里它一定是**未选中**的
// 用户明确要求：**选完一个人就清空输入框 + 光标回来**（见上面的说明）
search.value = '';
applyFilter();
refresh();
search.focus();
} else {
// 单选：空格 = 选中并关闭（与"点一个人"同义）
fbox.checked = true;
refresh();
closeModal(true);
}
return;
}
// 回车 = 确定（多选；单选没有确定这一步，浏览器里没有可提交的表单，
// 所以回车**不**做任何事，避免"不小心把弹窗关了却没选上"）
if (e.key === 'Enter' && stagedMode) {
e.preventDefault();
refresh();
closeModal(true);