v50:旅程题图裁剪修好(AJAX 片段漏绑)/ 修正表单换最新控件+取消 / 打标签误报修复 / 名册按钮编辑蓝删除红
用户 2026-10-14 四条。
① 旅程详情点点「编辑」,里面的图片改不了、拖入/选中都不弹裁剪框
用户原话:「应该弹出来一个让我裁剪的那个框。但是现在却没有那个裁剪的框。」
**真根因**(assets/js/app.js):裁剪那段 IIFE 是**首屏一次性**的 ——
`var inputs = querySelectorAll('[data-crop-ratio]'); if (!inputs.length) return;`
在脚本加载那一刻扫一遍,扫不到就整段放弃。而「编辑旅程」是 **AJAX 取片段、
innerHTML 塞进弹框**的(templates/trips/show.php 的 loadInto)⇒ 详情页首屏
**没有**题图框 ⇒ 当场 return ⇒ 片段里的框**从来没有 change 监听**
⇒ 拖入/选中什么都不发生。
修法:拆成 `bindOne(input)`(幂等,打 `data-crop-bound` 标记)+ `scan(scope)`;
首屏调一次 + **MutationObserver 补挂**;库的按需加载改成"首次真要用时才拉"。
⚠️ 与「拖动上传」那段(bindInput/scan/MutationObserver)**同一手法** ——
它早就解决了"动态插进来的文件框也要接得住",裁剪这段当年漏了这层。
② 旅程「修正」表单(templates/trips/_revise_form.php)
用户三条:「开始和结束时间…应该改成我们最新的控件样式」「上面记录实际执行情况
那一段话说明也不要了」「记录这次修正的按钮应该放在右边,另外应该加一个取消」。
· 起止换 `$dtPlugin = 'antd'`(与 node_form.php 同款;当年 v41 刻意没换,现按用户要求统一)
· 删 show.php 里 `.rev-tip` 整段
· 按钮行 `rev-foot` 靠右 + 新增 `[data-rev-cancel]`;取消由 show.php 的**事件委托**
处理(`details.rev-open` 收起)—— 不用内联脚本(片段会被 AJAX 换进来)
③ 名册打标签误报「这个人不在你的名册里」
用户原话:「我搜索出来一个人…给他打标签,它会显示说这个人不在我的名册里,
其实他是在我的名册里的。」
**真根因**(index.php 的 people/tags):`isset(PersonRoster::visibleIds(...)[$pid])`,
而 `visibleIds()` 返回的是 **id 值列表** `[2,3,4,5]`(键是 0,1,2,3)——
把值列表当 id→值 映射用。只有 `$pid` 恰好等于数组下标时才为真:名册里人少时
(下标<pid)**谁都判"不在名册"**;一般情况只有"第 pid+1 个"碰巧过。
修法:改用 `in_array(..., true)`(与同文件 `inMyRoster` 写法一致)。
⚠️ **没改 `visibleIds()` 的返回形状** —— 另一处调用已是正确的 in_array,改形状会
静默弄坏它。陷阱:`PersonGroup::visibleIds()` 是集合、`PersonRoster::visibleIds()`
是值列表,**同名不同形状**,已在两处注释里写明。
(附带确认:不在名册的人本来就不出现在列表里,用户"不在名册就该搜不出来"本就成立。)
④ 名册列表按钮:编辑蓝底、删除红底
用**作用域类** `.person-edit`(var(--blue)) / `.person-del`(var(--red)) 三处形态都写,
不动全局 .btn-edit/.btn-danger 的语义(v41 踩过"裸类改色波及影院管理页")。
hover 在这里补 `filter: brightness(.92)`(文末"全站悬停统一"段是白名单写的)。
验证(新增 2 套件 + 更新 1 处旧断言):
· _fb_v50_ui.py(22 条真机断言:题图绑定/选中与拖入都弹裁剪框 16:9、修正 antd+无说明+
靠右+取消能收起、名册内 3 人打标签均成功且不误报、编辑蓝删除红)
· _b50_falsify.py(**10 注入 / 21 断言全抓**)
· _fb_e2e.py 的 ★第28-6条断言随 v50-4 升级(原查 var(--brand-soft),现查蓝+红作用域类)
· familybook_deploy.sh 加 v50 静态断言段(含两条"旧写法必须已不在"的反向断言)
· e2e 1511/0、几何、各批反向验证全绿;**v50 四条 22/0**、**v50 反向验证 21/0**;
全流水线 **PUSH_RC=0**,部署脚本 0 FAIL(含"第50批形态");线上 6 文件 md5 逐字节一致;
线上门禁诊断 16 项全 YES,临时页已删并复验 404
⚠️ 收尾时发现并修掉一个**自设陷阱**:我在 show.php 的删除说明里**整句引用了**用户点名要删的
那句原话 ⇒ 朴素 strpos 判"已删"时会命中我自己的注释(线上诊断 C8=NO)。
部署断言用 `_code_has`(剥注释)所以是对的,但这类字串留在注释里迟早坑到别的检查
⇒ 已改成不整句照抄。**改注释也改了文件哈希 ⇒ 又重跑一次全量部署**。
⚠️ 流水线末段唯一"失败"同 v49:沙箱安全删除门禁(按轮次累计删满 50 就终止进程);
单独重跑 _fb_v46_carddav.py = 29/0 ⇒ 环境护栏,与本次改动无关。
This commit is contained in:
@@ -417,11 +417,17 @@ foreach ($bysStage as $lst) { $shown += count($lst); }
|
||||
<details class="rev-open">
|
||||
<summary class="btn btn-warn btn-sm">修正</summary>
|
||||
<div class="rev-panel">
|
||||
<p class="rev-tip">
|
||||
记录实际执行情况。这仍与上面那条计划是<b>同一条记录</b>,
|
||||
原来的计划数据会保留在「修正记录」里,方便日后对比。
|
||||
点完这里<b>这段行程仍是计划</b>;确认无误后再点「完成」。
|
||||
</p>
|
||||
<?php /* 🔴 v50(用户 2026-10-14 第 3 条):
|
||||
「上面…那一段话说明也不要了。」
|
||||
⇒ 删掉原来那段 `.rev-tip`(开头是「记录…」、解释"仍是同一条记录"
|
||||
的那三行)。表单本身已经说清"实际的开始/结束""这次修正的说明",
|
||||
那段解释属于噪音。
|
||||
⚠️ 这段注释**刻意不整句照抄**用户点名的原话 ——
|
||||
部署断言要查"那句已删",若在注释里留下原文,
|
||||
那种朴素 grep 会命中注释而恒红(本项目已踩过多次)。
|
||||
要查"删没删"的断言请用 `_code_has`(剥注释)。
|
||||
⚠️ 别顺手删 `.rev-panel` / `.rev-open` 这两个类名 ——
|
||||
它们是展开区的结构锚点(CSS 与测试都用)。 */ ?>
|
||||
<?php
|
||||
$rvNode = $n; $rvTripId = (int)$t['id'];
|
||||
$rvPeople = $people ?? [];
|
||||
@@ -651,6 +657,19 @@ foreach ($bysStage as $lst) { $shown += count($lst); }
|
||||
return;
|
||||
}
|
||||
if (t.closest('[data-nf-close]')) { setOpen(false); return; }
|
||||
/* 🔴 v50(用户 2026-10-14 第 3 条):「应该加一个关闭/取消的按钮…不要再修正的话,
|
||||
应该有一个取消,不要让这个框显示了。」
|
||||
「修正」区是 `<details class="rev-open">`(原生可折叠,不用 JS 也能开)——
|
||||
⇒ 取消 = 把这个 details 的 `open` 去掉,回到收起状态。
|
||||
⚠️ 用委托(`closest`)而不是在 `_revise_form.php` 里内联脚本:
|
||||
那份片段会被 AJAX 换进来,内联脚本的位置与幂等都要额外照看;委托一次绑定就够。 */
|
||||
var rc = t.closest('[data-rev-cancel]');
|
||||
if (rc) {
|
||||
e.preventDefault();
|
||||
var det = rc.closest('details.rev-open');
|
||||
if (det) { det.open = false; }
|
||||
return;
|
||||
}
|
||||
var h = host();
|
||||
if (h && t === h) { setOpen(false); return; } // 点遮罩关;点面板内部不关
|
||||
});
|
||||
|
||||
Reference in New Issue
Block a user