Merged
Conversation
募集開始曜日に合わせて週が区切られていたため、カレンダー感覚と 合わず視認性が悪かった。常に月曜〜日曜の 7 列で揃うように変更し、 期間外セルは薄色＋クリック無効で、日別ビューへ遷移しないようにした。 - dateUtils に buildWeeklyGrid / getWeekStartDate / WeekStart を追加 (デフォルトは月曜起算、Props で日曜起算も選べる) - PC OverviewView を weekly grid ベースに刷新 - 期間外ヘッダーは opacity 0.35・cursor default、ボディは常に '—' - 週ラベルと (N 日) は期間内日付のみを参照 - MidWeekStart / SundayStart の Story と dateUtils のテストを追加
Contributor
🚀 Preview DeployApp: https://pr-344.dev-yps-crispy-carnival.pages.dev |
Contributor
📊 Playwright Test Report❌ テスト結果: 失敗 4 passed / 1 failed / 0 skipped
デプロイ先: https://github.com/yn1323/hosting-pages |
Contributor
📊 Playwright Test Report❌ テスト結果: 失敗 4 passed / 1 failed / 0 skipped
デプロイ先: https://github.com/yn1323/hosting-pages |
OverviewView は viewMode に関わらず display:none で常にマウントされる
ため、初回は shiftConfigAtom がデフォルト値 (dates:[]) で、weeks が空
配列 → useState(initialOpen) が {} を保持したままになる。dates 投入後
に weeks が再計算されても state は初期化時の空オブジェクトのまま残り、
open[wi] が undefined → WeekTable が描画されず <table> 要素が DOM に
出ない。結果として「一覧ビューで田中太郎のテーブルを見つける」E2E が
失敗していた。
open を「閉じているインデックスだけ false を持つ」意味に変更し、
未登録インデックスはデフォルトで開とみなすようにした。初期 state は
単に {} で済むため useMemo で initialOpen を構築する必要もない。
Contributor
🚀 Preview DeployApp: https://pr-344.dev-yps-crispy-carnival.pages.dev |
Contributor
📊 Playwright Test Report✅ テスト結果: 成功 5 passed / 0 failed / 0 skipped
デプロイ先: https://github.com/yn1323/hosting-pages |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
募集開始曜日に合わせて週が区切られていたため、カレンダー感覚と
合わず視認性が悪かった。常に月曜〜日曜の 7 列で揃うように変更し、
期間外セルは薄色＋クリック無効で、日別ビューへ遷移しないようにした。
(デフォルトは月曜起算、Props で日曜起算も選べる)