Files
argon-theme/.kiro/specs/pjax-lazyload-optimization/tasks.md

200 lines
6.8 KiB
Markdown
Raw Normal View History

# 实施计划PJAX 和 LazyLoad 优化
## 概述
本实施计划将 Argon 主题的 PJAX 和 LazyLoad 功能优化分解为离散的编码步骤。每个任务都基于设计文档,并引用具体的需求。
**实施策略:**
- 渐进式重构:优先修复关键问题,避免大规模重写
- 向后兼容:保持现有 API 和配置不变
- 测试驱动:每个功能模块都有对应的测试任务
## 任务
- [x] 1. 创建资源清理模块
-`pjax:beforeReplace` 中的清理逻辑封装到独立函数
- 创建 `cleanupPjaxResources()` 函数
- 包含 LazyLoad Observer、Zoomify、Tippy 的清理逻辑
- 添加错误处理和日志记录
- _需求2.3, 6.1-6.5, 7.1-7.5_
- [ ]* 1.1 为资源清理模块编写单元测试
- 测试 LazyLoad Observer 清理
- 测试 Zoomify 实例清理
- 测试 Tippy 实例清理
- 测试错误处理逻辑
- _需求2.3, 6.1-6.5, 7.1-7.5_
- [x] 2. 优化 LazyLoad 初始化函数
- [x] 2.1 添加 Observer 存在性检查
-`lazyloadInit()` 开头检查 `lazyloadObserver` 是否已存在
- 如果存在,先调用 `disconnect()` 再创建新实例
- _需求2.1, 2.2_
- [x] 2.2 创建优化的图片加载函数
- 创建 `loadImageOptimized()` 函数
- 使用 `requestAnimationFrame` 替代 `setTimeout`
- 实现 fadeIn 和 slideDown 效果
- _需求3.1, 3.2, 3.3_
- [x] 2.3 创建加载效果应用函数
- 创建 `applyLoadEffectOptimized()` 函数
- 使用双重 `requestAnimationFrame` 确保渲染同步
- 正确设置 CSS transition 和 transform 属性
- _需求3.1, 3.2, 3.3_
- [x] 2.4 实现滚动监听降级方案
- 创建 `lazyloadFallback()` 函数
- 使用节流函数优化性能
- 绑定 scroll 和 resize 事件监听器
- _需求2.4_
- [x] 2.5 优化图片状态清理
- 使用正则表达式移除所有 `lazyload-style-*` 类名
- 确保 `data-src` 属性被移除
- _需求3.5_
- [ ]* 2.6 为 LazyLoad 模块编写属性测试
- **属性 2: LazyLoad Observer 清理完整性**
- **验证:需求 2.1, 2.2**
- [ ]* 2.7 为图片加载效果编写属性测试
- **属性 5: 图片加载效果使用 requestAnimationFrame**
- **属性 6: 图片加载效果 CSS 属性正确性**
- **验证:需求 3.1, 3.2, 3.3**
- [ ]* 2.8 为图片加载失败处理编写属性测试
- **属性 7: 图片加载失败处理**
- **属性 8: LazyLoad 类名清理**
- **验证:需求 3.4, 3.5, 10.4**
- [x] 3. 优化 PJAX 事件处理器
- [x] 3.1 重构 `pjax:beforeReplace` 事件处理器
- 调用 `cleanupPjaxResources()` 函数
- 保留 UI 状态更新逻辑
- 保留滚动位置处理逻辑
- _需求2.3, 6.1-6.5, 7.1-7.5_
- [x] 3.2 重构 `pjax:complete` 事件处理器
- 移除重复的初始化调用
- 保持单次调用 `waterflowInit()``lazyloadInit()``zoomifyInit()`
- 添加错误处理包裹所有初始化函数
- _需求1.1, 1.2, 1.3_
- [x] 3.3 重构 `pjax:end` 事件处理器
- 移除 `setTimeout` 中的重复初始化调用
- 只保留 `resetMobileCatalog()``resetGT4Captcha()` 调用
- _需求1.1, 1.2, 1.3, 8.1, 8.2_
- [ ]* 3.4 为 PJAX 事件处理编写属性测试
- **属性 1: PJAX 初始化函数单次调用**
- **属性 3: PJAX beforeReplace 资源清理**
- **验证:需求 1.1-1.3, 2.3, 6.1-6.5, 7.1-7.5**
- [x] 4. 创建 GT4 验证码重置函数
-`pjax:end` 中的 GT4 重置逻辑封装到 `resetGT4Captcha()` 函数
- 重置状态变量(`geetestVerified``geetestAutoSubmitting`
- 清空隐藏字段
- 清空容器 DOM
- 调用初始化函数
- 添加错误处理
- _需求4.1, 4.2, 4.3, 4.4, 4.5_
- [ ]* 4.1 为 GT4 验证码重置编写属性测试
- **属性 9: GT4 验证码状态重置完整性**
- **属性 10: GT4 验证码初始化错误处理**
- **验证:需求 4.1-4.5**
- [x] 5. 优化滚动位置管理
- [x] 5.1 验证 `pjax:click` 事件中的滚动位置设置逻辑
- 确认分页链接的滚动位置计算正确
- 确认 Banner 封面模式的条件判断正确
- _需求5.1, 5.2, 5.5_
- [x] 5.2 验证 `pjax:complete` 事件中的滚动位置恢复逻辑
- 确认 `pjaxScrollTop > 0` 时恢复滚动位置
- 确认恢复后 `pjaxScrollTop` 被重置为 0
- _需求5.3, 5.4_
- [ ]* 5.3 为滚动位置管理编写属性测试
- **属性 11: 滚动位置恢复正确性**
- **属性 12: 分页链接滚动位置计算**
- **验证:需求 5.2-5.5**
- [x] 6. 检查点 - 确保所有测试通过
- 运行所有单元测试
- 运行所有属性测试
- 检查代码覆盖率(目标:≥ 80%
- 如有问题,询问用户
- [x] 7. 添加降级方案和兼容性检查
- [x] 7.1 为 IntersectionObserver 添加特性检测
- 确认 `'IntersectionObserver' in window` 检查存在
- 确认降级方案被正确调用
- _需求2.4_
- [x] 7.2 为第三方库添加存在性检查
- 检查 `typeof tippy !== 'undefined'`
- 检查 `typeof MathJax !== 'undefined'`
- 检查 `typeof renderMathInElement !== 'undefined'`
- _需求4.5_
- [ ]* 7.3 为降级方案编写属性测试
- **属性 4: IntersectionObserver 降级方案**
- **验证:需求 2.4**
- [x] 8. 代码审查和优化
- [x] 8.1 检查所有错误处理
- 确保所有 try-catch 块都有适当的日志记录
- 确保错误不会阻塞其他功能
- _需求4.5_
- [x] 8.2 检查代码风格
- 使用 Tab 缩进
- 使用单引号
- 使用严格相等 `===`
- 添加 JSDoc 注释
- [x] 8.3 优化性能
- 检查是否有不必要的 DOM 查询
- 检查是否有重复的计算
- 使用 `{passive: true}` 优化滚动事件监听器
- [ ] 9. 集成测试
- [ ] 9.1 测试完整的 PJAX 页面切换流程
- 测试单次切换
- 测试多次连续切换
- 测试快速点击多个链接
- [ ] 9.2 测试 LazyLoad 在 PJAX 后的工作状态
- 测试图片懒加载
- 测试加载效果
- 测试降级方案
- [ ] 9.3 测试资源清理
- 测试内存使用情况
- 测试事件监听器数量
- 测试 DOM 元素清理
- [ ] 10. 最终检查点 - 确保所有测试通过
- 运行所有单元测试
- 运行所有属性测试
- 运行所有集成测试
- 检查代码覆盖率
- 如有问题,询问用户
- [x] 11. 文档更新
- 更新代码注释
- 添加 JSDoc 文档
- 更新 CHANGELOG如果存在
- 记录优化前后的性能对比
## 注意事项
- 任务标记 `*` 的为可选测试任务,可以跳过以加快 MVP 开发
- 每个任务都引用了具体的需求编号,便于追溯
- 检查点任务确保增量验证
- 属性测试标注了对应的设计属性编号
- 所有修改都在 `argontheme.js` 文件中进行
- 保持向后兼容,不修改全局变量名和配置选项