[fix] 戻るボタンのテキストとアイコンの更新、ヘルプカルーセルの条件を修正 #350
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
対応Issue
(任意)Issue があれば番号を入れてください
例: resolve #123
概要
prize ページでヘルプを開くと、表示されていない「景品ボタン」の説明が出てしまう問題を修正
prize ページでは 2 枚目の説明を「戻るボタン」に切り替え、home では従来どおり「景品ボタン」の説明を表示
実装詳細
view-user/src/components/common/Header/Header.tsx
ルーティング router.pathname を参照し、intro.js のステップ2を出し分け
home 等: element: "#PrizesIcon" + page2_title/page2_txt
prize: element: "#BackIcon" + page2_title_back/page2_txt_back
view-user/src/components/Layout/Layout.tsx
prize ページのナビゲーションで表示する BackIcon に id="BackIcon" を付与(ヘルプのアンカー用)
ついでに不要な空白を整理(動作影響なし)
view-user/src/components/common/icons/BackIcon/BackIcon.tsx
BackIconProps を追加し id を受け取り、IconFramework へ伝播できるように対応
view-user/src/locales/ja.ts / view-user/src/locales/en.ts
「戻るボタン」用の文言キーを追加
helpDescription.page2_title_back / helpDescription.page2_txt_back
画面スクリーンショット等
任意(必要なら添付してください)
確認観点:
home: ヘルプ 2 枚目が「景品ボタン」で #PrizesIcon を指す
prize: ヘルプ 2 枚目が「戻るボタン」で #BackIcon を指す
テスト項目
[ ] home でヘルプを開くと 2 枚目が「景品ボタン」の説明になる
[ ] prize でヘルプを開くと 2 枚目が「戻るボタン」の説明になる
[ ] 英語/日本語の両ロケールで文言が正しく表示される
[ ] ヘッダーのヘルプ起動/完了時のスクロール制御に副作用なし
[ ] ナビゲーションのアイコン配置が崩れていない
[ ] ESLint がエラーなく通る(警告は許容範囲であることを確認)
備考
初回アクセス時のみヘルプが自動起動する(localStorage.isStartIntrojs)。再確認時は該当キーの削除が必要です。
仕様観点で「戻るボタン」の文言は調整可能です。必要があればお知らせください。