BoxPistols / BoxPistols/LP-WebDesign-Brand

コードレビュー指摘の軽微な修正一式(CodeRabbit検証済み6件)

Open
#23 0 comments 0 reactions 1 assignee Claimed by @BoxPistols View on GitHub
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.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.