mirror of
https://github.com/hectorqin/reader.git
synced 2026-11-11 21:41:02 +08:00
用户报的八条里,这一提交覆盖了除图标对齐之外的七条。每一条都先做成能复现的 测试或量出来的数字,再改。 ## 2. txt 没有处理就渲染,没有默认样式,也无法调整 三层都缺,所以修了三层。 服务端新增 `chapter-html:<n>`,把纯文本渲染成带 p 的片段 (`indexer/formats/text-html.ts`)。空行断段;「上一行以句末标点结尾、下一行以 句子开头」也断段——抓取来的网文常常每行一段且没有空行,少了这条整本书是一块。 章节标题行提升成 h3,否则正文会变成「第二章 落雨雨来了。」。引用仍是 `chapter:<n>`(它同时是阅读位置的标识,改它会把现有读者打回第一章),新增 `format` 字段说明取哪一份;`chapter:` 的 256KB 截断也不再作用于这份渲染结果—— 那个截断会把长章节切一半。 客户端新增 `TXT_STYLESHEET`,只在 `format === 'txt'` 时注入 shadow root,作用域 `.txt-body`,所以 EPUB 作者自己的样式一点不受影响。默认两字符首行缩进,用 `2em` 而不是 `2rem`:缩进要跟字号缩放。 阅读设置新增「正文」一节(段落缩进 / 段间距 / 编码),只在 TXT 上出现,前两项 是写在 stage 上的自定义属性,所以调整不需要重新注入章节。 ## 3. 翻页异常,往前翻页还会来回跳 `previous()` 用 `scrollTop > 8` 判断「还有没有地方可退」,`next()` 用 `scrollTop < limit - 8` 判断「还有没有地方可进」,然后两边都以 0.9 屏步进。 这三个数字互不相容:forward 落到的位置,back 不认为那是边界。读者看到的就是 来回跳。 改成按**页序号**算术:整屏步进,页边界由滚动**范围**(`scrollHeight - clientHeight`)而不是内容高度推出。这一个区别是页数和可翻次数对不上的原因—— 2.31 屏的一章,按内容高度算 3 页并请求 offset 0/723/1446/2169,而 2169 会被 夹到 1674,也就是第 2 个边界,于是第四次按下的位置计数器不认。 `stepSection` 改成 await:它在跨章时是 fire-and-forget,而 `turnPage` 在 `previous()` 返回后就播放动画——动画演的是读者刚离开的那一章。 ## 4. sync 接口会不停的重复渲染,sync 接口调用太过频繁 两处。`onPosition` 每个位置调三次 `patch()`,而 `patch` 会重绘整棵 chrome 树,滚动时每帧三次;合并成一次。 `flushProgress` 无条件 `syncNow()`,于是 1.5 秒一轮 POST+GET,读者只要在读就 一直发。`SyncEngine.schedule()` 去抖并**保证尾次**——没有尾次的话,读者翻完 最后一页就关书,那一页永远发不出去。 ## 5/6. 布局样式,以及目录/设置改半屏 面板从 `inset: 0` 改成底部半屏 + 遮罩 + 把手,上半屏正文仍然看得见:选章节、 调字号都是对下面那一页的调整,全屏让读者为了做调整而离开那一页。 状态行从 flex 列里的一行改成浮在正文上的药丸。横条每出现一次就把正文推下去 一行再收回来,也就是把读者正在读的那句挪走。`ui:review` 有一条断言盯着它。 页脚改成两行不同职责:控制行居中、读数是三个固定位置的等宽数字。目录行改成 「序号 + 标题」,当前章高亮一次而不是每行都划线;当前窗口到不了的行标记出来 而不是静静地没反应。 ## 7. 点击页面的事件处理层次 一次触摸只属于一个目标,最里面那个赢。两个真缺陷: - 必须用 `composedPath()` 而不是 `event.target`。书的正文在 shadow root 里, shadow 边界会**重定向**事件,所以 target 是 shadow host——用 target 写不出 「段落」和「同一个 shadow 树里的按钮」的区别。 - **阅读面自己是个滚动容器**。滚动模式就是让 shadow host 滚起来,于是「可滚动 的祖先拿走手势」这条规则让它吃掉了每一个手势:翻页层级完全正确,但永远 到不了。这是「点击页面没反应」的真正原因。 另外,按到控件的触摸不再被吞掉 click——之前工具栏按钮可见、可点、没反应。 面板打开时它拥有点击:旁边露出的那一条正文不翻页。 ## 8. 端到端测试要做 UI 评审 新增 `web/tools/ui-review/`:真实 Chromium(390×844、2x)加载**生产产物**, 对着替身 API 逐屏截图并测量。截图给人看(「只是丑」不是测试失败,但是缺陷), 尺寸给流水线断言(面板不遮住整页、留出 ≥120px 正文、有遮罩和把手、状态提示 不推动正文、控件 ≥28px 且有名字、没有横向溢出)。接进 `client-ui-review` 流水线,只在界面相关文件变化时触发。 这套工具第一次跑就发现了四个真缺陷:`api.toc()` 返回形状不符时 `openStaged` 用 TypeError 打死整个 open(现在回退到 manifest 自己的条目);状态行一旦进入 loading 就再也清不掉(`hideStatus` 的守卫挡住了唯一的出口);翻页步进的 0.9 屏; 以及上面那条阅读面吃掉手势。 ## 测试 web 351 项、server 206 项全过。新增:`gestures.test.ts`(事件层级优先级)、 `reader-paging.test.ts`(页序号算术,含子像素取整与滚动范围)、 `mount.test.ts`(单次渲染与卸载后丢弃更新)、`text-html.test.ts`(分段、 转义、标题提升、不截断)。
79 lines
3.1 KiB
TypeScript
79 lines
3.1 KiB
TypeScript
// @vitest-environment jsdom
|
|
import { describe, expect, it } from 'vitest';
|
|
import { h } from 'preact';
|
|
import { mountUI } from '../src/ui/mount.ts';
|
|
|
|
/**
|
|
* The bridge between a screen's state bag and the component tree.
|
|
*
|
|
* Worth its own file because the two properties below are what make the reader's
|
|
* chrome affordable to update from a scroll handler, and both are easy to lose in a
|
|
* refactor that looks equivalent.
|
|
*/
|
|
describe('mountUI', () => {
|
|
it('renders once on mount and once per update, not once per field', () => {
|
|
// A `patch()` is a full tree diff. A screen that calls it three times for one
|
|
// logical change pays three times, and the reader's position handler used to do
|
|
// exactly that — once per frame while a finger was down.
|
|
const container = document.createElement('div');
|
|
let renders = 0;
|
|
const ui = mountUI(
|
|
container,
|
|
(value) => {
|
|
renders += 1;
|
|
return h('p', null, String(value));
|
|
},
|
|
0,
|
|
);
|
|
expect(renders).toBe(1);
|
|
|
|
// One `update` per logical change is the contract; a caller that batches its
|
|
// fields into one object gets one render no matter how many fields it carries.
|
|
ui.update({ a: 1, b: 2, c: 3 });
|
|
ui.update({ a: 2, b: 2, c: 3 });
|
|
expect(renders).toBe(3);
|
|
});
|
|
|
|
it('drops updates after unmount instead of repainting a cleared container', () => {
|
|
// Every screen loads asynchronously and can be left mid-load. Without the
|
|
// guard, a late response paints a whole screen into a container Preact has
|
|
// already been told is empty — and a later `render(null)` does not remove it,
|
|
// because as far as Preact is concerned it was never its child.
|
|
const container = document.createElement('div');
|
|
const ui = mountUI(container, (value) => h('p', null, String(value)), 'a');
|
|
ui.unmount();
|
|
ui.update('late');
|
|
expect(container.textContent).toBe('');
|
|
});
|
|
|
|
it('keeps a node the caller owns across renders, via a ref', () => {
|
|
// How the reader puts its `stage` in the tree: not as a child the diff would
|
|
// recreate, but through a ref callback that appends the element the screen has
|
|
// kept since its constructor. A tree that replaced the node would take the
|
|
// shadow root, the scroll position and the paginator's measurements with it —
|
|
// and the position, because `ReaderView` measures *that* element.
|
|
const container = document.createElement('div');
|
|
const stage = document.createElement('div');
|
|
stage.className = 'stage';
|
|
const ui = mountUI(
|
|
container,
|
|
(value) =>
|
|
h(
|
|
'div',
|
|
{ className: 'wrap', 'data-v': String(value) },
|
|
h('div', {
|
|
className: 'slot',
|
|
ref: (node: Element | null) => {
|
|
if (node && stage.parentElement !== node) node.append(stage);
|
|
},
|
|
}),
|
|
),
|
|
0,
|
|
);
|
|
expect(container.querySelector('.stage')).toBe(stage);
|
|
ui.update(1);
|
|
expect(container.querySelector('.stage')).toBe(stage);
|
|
expect(container.querySelector('.wrap')?.getAttribute('data-v')).toBe('1');
|
|
});
|
|
});
|