BoxPistols / BoxPistols/LP-WebDesign-Brand
コードレビュー指摘の軽微な修正一式(CodeRabbit検証済み6件)
- Dominant language
- JavaScript
- Stars
- 1
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Description
## 背景
PR #11 のCodeRabbitレビューから、実コード照合で「実在するが軽微(データ損失・クラッシュなし)」と確認した項目をまとめたもの。マージ判断はブロックしなかったが、フォローアップとして修正する。
## 項目
- [ ] `getInlineCSS()` の2箇所のfetchで `r.ok` チェックが無い(`js/generator.js`)。読込失敗時にエラーHTMLがCSSとして埋め込まれる可能性
- [ ] `loadProjectById()` でプロジェクト読込後に `saveState()` が呼ばれておらず、undo履歴にチェックポイントが残らない(`importProjectFromJSON` は呼んでいるので同じフローに揃える)
- [ ] `js/lp-motion.js` の `_onScroll()` が `global.innerHeight` を常に使用しており、`scrollTarget` が要素(例: previewFrame)の場合にパララックス計算が不正確になりうる
- [ ] `js/lp-motion.js` の `_runCounter()` が独自の `requestAnimationFrame` ループを回しており、`this._rafId` で追跡されていないため `destroy()` 時にカウンターアニメーションがキャンセルされない
- [ ] `js/archetype-templates.js` のSVG `clipPath`/`filter` ID(`lpjpav1`, `lpg-e1`, `lpg-s1` 等)が固定値。同一テンプレートを複数配置するとDOM内でID重複(描画は崩れないがHTML仕様違反)
- [ ] `docs/MOBIRISE_ROADMAP.md` 内の「F2」という誤字(正しくは「F0」)+Markdown lint警告(MD022見出し前後の空行、MD058テーブル前後の空行)、CSSの `currentColor`→`currentcolor` 表記統一(Stylelint)
## 参照
PR #11 CodeRabbitレビュー(4件のレビューラウンド)
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.