Skip to content

職務経歴書フォームの項目削除を見出しのアイコンボタンへ移設#303

Merged
yusuke0610 merged 1 commit into
mainfrom
feat/resume-form-item-delete-icons
Jun 5, 2026
Merged

職務経歴書フォームの項目削除を見出しのアイコンボタンへ移設#303
yusuke0610 merged 1 commit into
mainfrom
feat/resume-form-item-delete-icons

Conversation

@yusuke0610

@yusuke0610 yusuke0610 commented Jun 5, 2026

Copy link
Copy Markdown
Owner

概要

職務経歴書フォームで、各項目の削除ボタンが項目の最下部にフル幅で縦積みされており、
スクロールすると「どの削除ボタンがどの項目のものか分かりづらい」「縦に長い」問題を解消する。

削除を各項目の見出し右端の小さなゴミ箱アイコンに移設し、追加ボタンはプラスアイコン+テキストに調整した。
対象は職務経歴 / 取引先 / プロジェクト / 資格の4セクション。

注: 当初 #302fix/resume-pdf-format-and-form)ベースの stacked PR だったが、#302 が main にマージされたため最新 main へ rebase し、base を main に変更済み。差分は本コミット1件のみ。

変更内容

新規

  • components/ui/DeleteIconButton.tsx — 見出し右端に置くアイコンのみの削除ボタン(4箇所で共通利用)。
    stopPropagation でアコーディオンの誤トグルを防止。
  • components/icons/TrashIcon.tsx / PlusIcon.tsx — カスタム SVG アイコン(依存追加なし)。

各セクション

  • 職務経歴: Collapsible の既存 headerActions スロットに削除アイコンを配置。
  • 取引先: 取引先見出し行の右端に削除アイコン(休業時は aria-label を「休業を削除」に切替)。
  • プロジェクト: 既存の行内「編集/削除」のうち削除のみアイコン化(編集は維持)。
  • 資格: 各行を flex 化し右端に削除アイコンを配置。
  • 各「追加」ボタンに PlusIcon を付与(位置は従来どおり)。

その他

  • 削除ロジック・props のシグネチャは変更なし(配置と見た目のみ)。
  • アイコンの aria-label は UI_MESSAGES に集約。

テスト

  • 最新 main へ rebase 後にローカル CI 相当すべて pass: ESLint / build(tsc + vite)/ vitest 208 passed(新規 DeleteIconButton 3 ケース含む)/ node:test / lint-frontend-messages
  • ブラウザでサンプルデータをレンダリングし、各見出しの削除アイコン・追加アイコンの表示を目視確認済み
  • 新規ルート/認証/レイアウト変更は無いため E2E トリガー対象外

Generated by Claude Code

@coderabbitai

coderabbitai Bot commented Jun 5, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@yusuke0610, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 24 minutes and 45 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 2845bd5e-8ad3-49c9-9a2b-3a0794658330

📥 Commits

Reviewing files that changed from the base of the PR and between b2bb498 and d83291a.

📒 Files selected for processing (11)
  • frontend/src/components/forms/CareerFormEditors/CareerExperienceEditor.tsx
  • frontend/src/components/forms/CareerFormEditors/ClientEditor.tsx
  • frontend/src/components/forms/CareerResumeForm.module.css
  • frontend/src/components/forms/sections/CareerExperienceSection.tsx
  • frontend/src/components/forms/sections/CareerQualificationsSection.tsx
  • frontend/src/components/icons/PlusIcon.tsx
  • frontend/src/components/icons/TrashIcon.tsx
  • frontend/src/components/ui/DeleteIconButton.module.css
  • frontend/src/components/ui/DeleteIconButton.test.tsx
  • frontend/src/components/ui/DeleteIconButton.tsx
  • frontend/src/constants/messages.ts
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/resume-form-item-delete-icons

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

職務経歴・取引先・プロジェクト・資格の各項目で、最下部にフル幅で縦積み
されていた削除ボタンが「どの項目のものか分かりづらい」問題を解消する。
削除を各項目の見出し右端の小さなゴミ箱アイコンに移設し、追加ボタンは
プラスアイコン+テキストに調整した。

- 共通コンポーネント DeleteIconButton を追加(4箇所で再利用、stopPropagation
  でアコーディオンの誤トグルを防止)
- カスタム SVG アイコン TrashIcon / PlusIcon を追加(依存追加なし)
- 職務経歴は Collapsible の既存 headerActions スロットを利用
- 削除ロジックと props のシグネチャは変更なし(配置と見た目のみ)
- aria-label は UI_MESSAGES に集約

https://claude.ai/code/session_01RkNhFS1GApCyYXvM64zPcZ
@yusuke0610
yusuke0610 force-pushed the feat/resume-form-item-delete-icons branch from 4b2e8dc to d83291a Compare June 5, 2026 02:05
@yusuke0610
yusuke0610 changed the base branch from fix/resume-pdf-format-and-form to main June 5, 2026 02:05
@yusuke0610

Copy link
Copy Markdown
Owner Author

@coderabbitai PVして

@yusuke0610 yusuke0610 closed this Jun 5, 2026
@coderabbitai

coderabbitai Bot commented Jun 5, 2026

Copy link
Copy Markdown

@yusuke0610 了解しました!PR #303 のレビューを開始します 🐇

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@yusuke0610 yusuke0610 reopened this Jun 5, 2026
@yusuke0610

Copy link
Copy Markdown
Owner Author

@coderabbitai PVして

@coderabbitai

coderabbitai Bot commented Jun 5, 2026

Copy link
Copy Markdown

@yusuke0610 かしこまりました!PR #303 のレビューを開始します 🐇

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@yusuke0610
yusuke0610 merged commit b570310 into main Jun 5, 2026
18 checks passed
@yusuke0610
yusuke0610 deleted the feat/resume-form-item-delete-icons branch June 10, 2026 05:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants