|
|
25eacc1c3c
|
refactor(lightbox): modernize to const/let, template literals, drop IIFE
纯现代化重构,行为不变(23 个回归测试全绿作证):
- var → const/let(state、gotoIndex 的 newIndex 用 let,其余 const)
- 循环 for+var+IIFE包装 → for..of/forEach + const(天然捕获迭代变量,
消除旧 var 循环闭包必须立即执行函数固定变量的写法)
- 字符串拼接 → 模板字面量(counter 文本、transform、width/height)
- 事件处理 function() → 箭头函数(无 this 依赖场景)
- 拆掉外层 (function(){'use strict'; ... })():ES module 作用域已封装,
IIFE 是历史遗留;辅助函数仍用 function 声明(openLightbox↔closeLightbox
互递归依赖提升)
几何纯函数(fitCentered/transformFor/originalUrl)已在 geometry.ts,
本文件只剩 DOM 胶水 + state 单例。
|
2026-06-24 11:44:06 +08:00 |
|
|
|
aadfed2595
|
test(lightbox): add happy-dom behavior regression tests for refactor
为现代化重构(var→const、拆 IIFE、for→for..of)补行为回归防线。
黑盒驱动 window.__initLightbox,覆盖高风险路径:
- 循环闭包捕获 idx(点第 1/2/3 张 counter 显示正确序号)
- 点击打开(overlay 创建、origSrc 去 query、caption、单张 counter 隐藏)
- 关闭(Esc / 点背景;点图片本身不关)
- 图集 gotoIndex 循环边界(首末衔接 + originNode 更新后焦点归还)
- 单张图不参与切换
happy-dom 20.10.6 作 devDep,vitest 默认环境改 happy-dom;
geometry.test.ts 用 // @vitest-environment node 指令保持纯函数在 node 跑。
23 tests (12 geometry + 11 behavior) 全绿。
|
2026-06-24 11:40:36 +08:00 |
|
|
|
507d271a2b
|
fix(lightbox): prevent page scroll on close via focus({ preventScroll: true })
关闭灯箱时 removeOverlay() 调用 .focus() 归还焦点给原图,浏览器的 focus()
默认会 scrollIntoView,导致页面自动滚动把原图完整纳入视口。用户向下滚动后
只点露出的下半截图、再 Esc 关闭时,页面会向上跳到整图可见——非预期行为。
focus({ preventScroll: true }) 抑制该滚动(Chrome68+/FF68+/Safari15.4+ 支持),
焦点归还的无障碍语义不变。
|
2026-06-24 11:27:15 +08:00 |
|
|
|
b4ef906141
|
refactor(lightbox): move .lightbox-* CSS into lightbox project, wire import
|
2026-06-24 11:02:44 +08:00 |
|
|
|
5f57d375ea
|
feat(lightbox): migrate DOM logic to TypeScript index.ts with self-start contract
|
2026-06-24 10:57:23 +08:00 |
|
|
|
44fc641aaa
|
feat(lightbox): extract geometry pure fns with vitest coverage
|
2026-06-24 10:52:48 +08:00 |
|