用户 9 条原话要点:
①「观影模块,编辑观影,已经保存的海报和票根、合影,都没带进来。」
②「个人设置页面,"只影响你自己看到的样子"这个提示删除,不要了。我怎么称呼家人,
这个卡片,不要了。我的资料这个卡片,不要了。」
③「旅程模块,编辑行程,点击保存没有反应。」
④「所有上传图片视频和附件的模块,都允许多次拖动添加文件上传,拖动一个进去,
就开始上传,还可以继续拖动,继续上传,最后统一保存即可。」
⑤「添加行程,线路名、承运方,都不要了。车辆,只在出行方式选择自驾时才出现,
其他情况下默认不出现。出发地和到达地,应该在同一行。」
⑥「日程模块,周视图和日视图,应该在视图中间显示一条红线,用来表示当前进行的时间…
用一个不断从上到下的红线,来表示当前时间在视图上的刻度和位置。」
⑦「对于重复的日程,当点击删除的时候,应该提示是删除这一次,还是全部都删除,
还是删除这一次及以后的。」
⑧「日程的提醒,是可以多选的,就是说,一个日程,我可以选择让它 5 分钟提醒,
10 分钟提醒等等。」
⑨「很多按钮的悬停效果非常奇怪,感觉纯粹是换了一个背景色…应该调整所有按钮的悬停效果,
保持统一,但是不要单纯的换一个不协调的背景色。」
🔴 三个真根因(都是"看着不像 bug"的地方):
1. **③ 点保存没反应 = `<form>` 嵌套**(HTML 非法):`trips/node_form.php` 的照片缩略图上
套了 `<form action=photo_del>`,而整块在**外层行程表单内部**。浏览器解析到内层
`</form>` 时**把外层表单提前关掉** → 「保存」按钮**掉出表单**(真机实测
`button.form === null`)。源码里完全看不出,**只有真浏览器量 `btn.form` 才现形**;
且**只有这段行程已经有照片时才触发** —— 所以表现为"有时候好、有时候坏"。
→ 按钮留在原位(布局不动),用 HTML5 `form="id"` 关联到**表单外**的独立表单。
2. **⑤ 车辆藏不住(第 17 批就写过、一直没生效)**:`.field` 自己声明 `display:flex`,
作者样式压过浏览器默认的 `[hidden]{display:none}` → `hidden` 形同虚设。
**本项目第 4 次栽同一个坑**(.att-item / .pv / .pp-item 各一次)→ 补 `.field[hidden]`。
3. **④ 拖动即时上传的卡片"看不见"**:模板给 `.stage-list` 加了 `hidden`(没人上传时
不留空边框),JS 建卡片时忘了 `list.hidden = false` → 卡片在 DOM 里但整条清单
`display:none`、尺寸 0×0。**HTTP 层断言全绿**(HTML 里确实有卡片),
**只有真机量尺寸才发现** —— 由新写的真机探针抓到。
本批新增能力:
- **拖动即时上传**(第 4 条):新增 `upload_stage.php` + `app/StagedUpload.php` +
`templates/_stage_drop.php`。拖进来 → **立刻** POST 到暂存目录 → 回 `stoken` →
放进隐藏字段 `stage[]` → 保存时在 `attach_save_files()` 里**认领**(移文件 + 建附件行)。
`media.php` 加 `?stage=` 预览(拖完立刻看得见缩略图)。
⚠️ 三个必须踩准:**串行队列**(并发会让缩略图顺序乱跳)、**提交守卫**
(还有没传完就 preventDefault,否则"拖了但没保存上"必然发生)、**卡片清单要 un-hide**。
⚠️ 海报/票根走**独立字段名** `stage_poster`/`stage_ticket`(各自单槽位、要按比例重裁)。
⚠️ 落点**就近取**(`input.closest('.field')`):一个表单里多个上传区不能串到第一个去。
- **观影编辑带出已存图片**(第 1 条):编辑页显示海报 / 票根 / 合影 + 各自的删除入口;
新增 `movies/media_del`(只删不换也能删,槽位走白名单);`movies/update` 收编辑页
新选的照片时**合并**而不是替换(否则一编辑就把已有合影冲掉)。
- **行程表单**(第 5 条):去掉线路名 / 承运方(**库里的值保留**:UPDATE 时不参与,
与 seat_info/gate 同一套);出发地/到达地包进 `.loc-row` 真同一行;车辆按自驾显隐。
- **当前时间红线**(第 6 条):位置在服务端算(与 `.tg-block` 同一套"24h = 100%"坐标),
前端每分钟自己挪(对齐到下一分钟整再起定时器),跨午夜 reload;只画在**今天那一列**。
- **重复日程删除三档**(第 7 条):与「编辑」那三档同名同序(全部 / 这一次及以后 /
只这一次)。「只删这一次」→ `skipOnce`(**绝不删整条系列**);「这一次及以后」→
老系列截 UNTIL,**必须清掉 COUNT**(两者并存时 COUNT 优先,不清就截不断)。
- **提醒多选**(第 8 条):同列存逗号串(`5,15,60`);Schema 把它从 **INT 改 VARCHAR**
(INT 存逗号串会被隐式转换,多档会静默只剩第一档);ICS 导出吐**多个 VALARM**。
- **悬停统一**(第 9 条):原来 14 种互不相干的 hover 底色 → 统一成**两套手法**:
有底色 `filter: brightness(.92)` 同色系加深 + 轻抬升;描边白底 `--hov-veil` 中性薄雾
+ 边框微显。放在 CSS **最末**用一条兜底收口(新按钮自动继承),并把十几条旧规则
清成"交给统一段"。`person_picker.js` 同时改为按需加载。
验证(本地全套):
- e2e **1004/0**(+21 条本批断言)· 升级 149/0 · 桌面几何 428/0 ·
**真机探针 25/0**(连续拖两次 / 红线位置与颜色与几何 / 车辆显隐 / 出发到达同行)。
- **反向验证 5/5**:①删除按钮改回嵌套 form ②卡片清单不 un-hide ③观影编辑不渲染海报
④车辆隐藏失效 ⑤「只删这一次」做成删全部 —— 五种坏法全部被抓住,还原后复绿 1004/0。
过程中还**抓到两条弱断言**(只查 DOM 有无、不查可见性/是否真渲染),已一并加强。
- 部署脚本静态断言段本地空跑 **305 条全过**。
170 lines
9.8 KiB
PHP
170 lines
9.8 KiB
PHP
<?php
|
||
// ⚠️ 「系统设置」**不在**顶栏里(用户明确要求):它已经收进右上角头像下拉菜单里了,
|
||
// 顶栏留「首页 / 家谱 / 名册 / 观影」四个日常入口,别再把管理入口顶出来。
|
||
// ⚠️ v11:「影院」也**不在**顶栏里了(用户明确要求)—— 影院是观影模块的一部分,
|
||
// 统一从观影列表页右侧的「设置」按钮进(那里还挂着高级搜索/统计/年度报告)。
|
||
// 别再往 $navItems 里加子功能入口,顶栏只放"模块"这一级。
|
||
// ⚠️ v14 起导航条**不再写死**:模块清单在 App\Module::LIST 里(加一个模块只改那一处),
|
||
// 显示哪些、什么顺序由**每个人自己**的 user_settings('nav_modules') 决定
|
||
// (用户第 10 条:允许设置哪些模块出现在导航条上,拖动排序)。
|
||
// App\Module::navFor() 内部做了两道兜底(剔除已不存在的模块、补上新装的模块),
|
||
// 这里直接拿结果用,不要再自己过滤一遍。
|
||
$navKeys = \App\Module::navFor((int)\App\Auth::personId());
|
||
$navItems = [];
|
||
foreach ($navKeys as $k) {
|
||
$navItems[$k] = [\App\Module::name($k), \App\Module::route($k)];
|
||
}
|
||
// 本页属于哪个模块 → 决定主题色(用户第 11 条:每个模块一个主题色)
|
||
$moduleKey = \App\Module::ofPage((string)$pageKey);
|
||
if ($moduleKey === '') {
|
||
$moduleKey = 'home';
|
||
}
|
||
$themeVars = \App\Module::cssVars($moduleKey, (int)\App\Auth::familyId());
|
||
// ⚠️ 变量名不要叫 $me:layout 是在 extract($data) **之后**被 require 的,
|
||
// 这里再赋值会把页面传进来的 $me(当前登录者的完整记录)覆盖掉。
|
||
// home.php 用 $me 画「补充亲属关系」链接,被覆盖成 true 之后
|
||
// 链接就变成 people/show&id=0,还顺带在页首打出一条 Warning。
|
||
// 改名前请先确认所有模板都没有依赖这个布尔值。
|
||
$isLoggedIn = \App\Auth::check();
|
||
?>
|
||
<!DOCTYPE html>
|
||
<html lang="zh-CN" class="no-js">
|
||
<head>
|
||
<?php /* v24 第 12 批 #2:解决「打开页面先闪一下原生输入框,然后才变成 Semi 控件」。
|
||
做法是经典的 **no-js / js 二态**:
|
||
· 服务端先渲染 class="no-js"(此时原生控件可见 = 无 JS 也能用);
|
||
· 下面这段内联脚本**在 <head> 里立即把 class 换成 js** ——
|
||
它在 CSS 生效前、任何渲染之前就跑了,所以浏览器**根本没机会**画出原生框;
|
||
· CSS 里 `.js .dt-input .dt-native { visibility:hidden }` 先把原生藏住
|
||
(用 visibility 而不是 display:none:保留占位高度,避免升级瞬间布局跳动);
|
||
· app.js 的 Semi 就绪后加 `.is-semi` 把 Semi 显示出来(原生继续藏着)。
|
||
这样"闪烁"从**结构上**消失,而不是靠调加载顺序去掩盖。 */ ?>
|
||
<script>document.documentElement.className =
|
||
document.documentElement.className.replace('no-js', 'js');</script>
|
||
<meta charset="utf-8">
|
||
<meta name="viewport" content="width=device-width, initial-scale=1">
|
||
<title><?= e($pageTitle) ?> · 家书</title>
|
||
<link rel="icon" href="/favicon.ico">
|
||
<link rel="apple-touch-icon" href="/apple-touch-icon.png">
|
||
<link rel="manifest" href="/site.webmanifest">
|
||
<meta name="theme-color" content="#A84A28">
|
||
<link rel="stylesheet" href="<?= e(asset('/assets/css/app.css')) ?>">
|
||
<?php /* v24 第 9 批 #9:Semi Design DatePicker 懒加载清单(用户点名 semi.design)。
|
||
这里只登记带版本号的资源地址,**不加载** —— app.js 只有在页面上真出现
|
||
.dt-input(templates/_datetime.php)时才去下这 4 个文件(合计约 4 MB,
|
||
绝不能让没有日期控件的页面白拉)。 */ ?>
|
||
<meta name="fb-semi" content="<?= e(json_encode([
|
||
'css' => asset('/assets/vendor/semi/semi.min.css'),
|
||
'react' => asset('/assets/vendor/semi/react.min.js'),
|
||
'dom' => asset('/assets/vendor/semi/react-dom.min.js'),
|
||
'ui' => asset('/assets/vendor/semi/semi-ui.min.js'),
|
||
], JSON_UNESCAPED_UNICODE | JSON_HEX_APOS | JSON_HEX_QUOT)) ?>">
|
||
<?php /* v24 第 11 批 #5:Cropper.js(裁剪)也是**按需加载** ——
|
||
只有页面上真有 input[type=file][data-crop-ratio] 时才去拉(见 app.js)。
|
||
同样在这里登记带版本号的地址,绝不在没有裁剪的页面白拉。 */ ?>
|
||
<meta name="fb-cropper" content="<?= e(json_encode([
|
||
'css' => asset('/assets/vendor/cropper/cropper.min.css'),
|
||
'js' => asset('/assets/vendor/cropper/cropper.min.js'),
|
||
], JSON_UNESCAPED_UNICODE | JSON_HEX_APOS | JSON_HEX_QUOT)) ?>">
|
||
</head>
|
||
<?php /* 主题色:把 --brand / --brand-2 / --brand-ink 三个变量在 <body> 上换成本模块的色,
|
||
下面所有用到品牌色的地方(按钮、链接、页头高亮…)自动跟着变。
|
||
⚠️ 只覆盖这三个变量,**不碰** --paper / --ink 这些纸墨底色 ——
|
||
模块色是用来"分区"的,把整页背景也换了就不是家书了。 */ ?>
|
||
<body class="mod-<?= e($moduleKey) ?>"
|
||
style="--brand: <?= e($themeVars['brand']) ?>; --brand-2: <?= e($themeVars['brand2']) ?>; --brand-ink: <?= e($themeVars['ink']) ?>;">
|
||
|
||
<?php if ($isLoggedIn): ?>
|
||
<header class="topbar">
|
||
<div class="topbar-inner">
|
||
<a class="brand" href="<?= e(url('home')) ?>">
|
||
<img src="<?= e(asset('/assets/icons/icon-96.png')) ?>" alt="">
|
||
<span>家书</span>
|
||
</a>
|
||
<nav class="nav">
|
||
<?php foreach ($navItems as $key => list($label, $route)): ?>
|
||
<?php /* 高亮按「本页属于哪个模块」判,而不是 pageKey 全等 ——
|
||
子页(观影里的影院管理/统计/年度报告)也要让「观影」亮着。 */ ?>
|
||
<a class="<?= $moduleKey === $key ? 'on' : '' ?>" href="<?= e(url($route)) ?>"><?= e($label) ?></a>
|
||
<?php endforeach; ?>
|
||
</nav>
|
||
<div class="topbar-right">
|
||
<?php /* 姓名 + 头像 = 一个下拉菜单,而不是「点名字改资料」+旁边两个裸链接。
|
||
裸链接在窄屏只能靠 display:none 藏起来(一藏入口就没了),
|
||
收进菜单之后同一个入口在手机上也点得到。
|
||
用 <details> 实现:不需要 JS 就能展开/收起,app.js 只额外补
|
||
"点外面"和"Esc"两件事。 */ ?>
|
||
<details class="menu who-menu">
|
||
<summary class="who" title="我的">
|
||
<img src="<?= e(avatar_url($_SESSION['avatar'] ?? null)) ?>" alt="">
|
||
<span class="who-name"><?= e(\App\Auth::name()) ?></span>
|
||
<?php if (\App\Auth::isGuest()): ?><span class="badge">访客</span><?php endif; ?>
|
||
<i class="caret" aria-hidden="true"></i>
|
||
</summary>
|
||
<div class="menu-list">
|
||
<a href="<?= e(url('settings/me')) ?>">个人设置</a>
|
||
<?php if (\App\Auth::isAdmin()): ?>
|
||
<a href="<?= e(url('settings')) ?>">系统设置</a>
|
||
<?php endif; ?>
|
||
<a href="<?= e(url('auth/password')) ?>">修改密码</a>
|
||
<a class="menu-exit" href="<?= e(url('auth/logout', ['_csrf' => csrf_token()])) ?>">退出登录</a>
|
||
</div>
|
||
</details>
|
||
</div>
|
||
</div>
|
||
</header>
|
||
<?php endif; ?>
|
||
|
||
<main class="<?= in_array($pageKey, ['install', 'login'], true) ? 'container-narrow' : 'container' ?>">
|
||
|
||
<?php foreach (take_flash() as $f): ?>
|
||
<div class="flash">
|
||
<div class="flash-item <?= $f['type'] === 'err' ? 'err' : '' ?>"><?= e($f['message']) ?></div>
|
||
</div>
|
||
<?php endforeach; ?>
|
||
|
||
<?php require $tplFile; ?>
|
||
|
||
</main>
|
||
|
||
<?php /* 全局附件预览层(v24 第 3 条):图片 / 视频 / PDF 都在本页这一层里看,
|
||
右上角恒有下载按钮。内容由 assets/js/preview.js 按 data-prev 现填。 */ ?>
|
||
<div class="pv" id="pv" role="dialog" aria-modal="true" aria-hidden="true" aria-label="附件预览" hidden>
|
||
<div class="pv-head">
|
||
<span class="pv-name"></span>
|
||
<span class="pv-meta"></span>
|
||
<a class="pv-dl" href="#">下载</a>
|
||
<button type="button" class="pv-close" aria-label="关闭">✕</button>
|
||
</div>
|
||
<div class="pv-stage"></div>
|
||
<?php /* v24 第 19 批 用户第 6 条:左右箭头,直接看下一张。
|
||
⚠️ 箭头用 `hidden` 属性控显隐(见 preview.js 的 syncArrows)——
|
||
只有一个可预览项时不该出现(点了没反应比没有更糟)。
|
||
与 .pv 一样,自己声明了 display 的元素要藏就得补 [hidden] 规则。 */ ?>
|
||
<button type="button" class="pv-nav pv-prev" aria-label="上一张" hidden>‹</button>
|
||
<button type="button" class="pv-nav pv-next" aria-label="下一张" hidden>›</button>
|
||
</div>
|
||
|
||
<script src="<?= e(asset('/assets/js/app.js')) ?>"></script>
|
||
<?php /* v24 第 20 批:选人组件脚本改成**按需加载**(页面上真有 `.pp` 才拉)。
|
||
⚠️ 理由与 Cropper.js 同一套:`person_picker.js` 只在放选人组件的页面才有用,
|
||
而它并不小;每页都无条件带一份,等于让"没有选人组件"的页面白付流量。
|
||
⚠️ 判据用 `document.querySelector('.pp[data-pp]')`(组件根节点的真实标记),
|
||
**不是**含糊地找 "pp" 这两个字母(页面里 `pp` 太容易撞:分页、其它类名…)。
|
||
⚠️ DOM 必须已经解析到组件才判断 → 脚本放在 `</body>` 前(本文件本来就是),
|
||
所以这里能直接查;不需要 DOMContentLoaded 包一层。
|
||
⚠️ 无 JS 兜底不受影响:组件的 `<noscript>` 块本来就是给"脚本没跑"准备的。 */ ?>
|
||
<script>
|
||
(function () {
|
||
if (!document.querySelector('.pp[data-pp]')) { return; }
|
||
var s = document.createElement('script');
|
||
s.src = <?= json_encode(asset('/assets/js/person_picker.js')) ?>;
|
||
s.defer = true;
|
||
document.head.appendChild(s);
|
||
})();
|
||
</script>
|
||
<script src="<?= e(asset('/assets/js/filter.js')) ?>"></script>
|
||
<script src="<?= e(asset('/assets/js/preview.js')) ?>"></script>
|
||
</body>
|
||
</html>
|