TagEdit: CodeRabbitレビュー指摘の修正

[komagata]

概要

#9809 のCodeRabbitレビューで指摘された項目を修正します。

変更内容

1. キャンセルボタンのbutton化 — _tag_edit_button.html.slim のキャンセル要素を div から button[type="button"] に変更（キーボード操作・アクセシビリティ対応）
2. 空li要素の修正 — pages/index.html.slim で非admin時に空の li.page-header-actions__item が残る問題を admin_login? 条件で囲んで修正

Summary by CodeRabbit

リリースノート

• 新機能

  • タグ編集機能がアカウント所有者のみに表示されるようになりました。

• バグ修正

  • キャンセルボタンのマークアップを改善し、適切なボタン要素として機能するようになりました。

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 974e5a71-caf6-4d72-89d5-7043a6ef2d7a

📥 Commits

Reviewing files that changed from the base of the PR and between 21d5dd6 and 6ef8433.

📒 Files selected for processing (2)

• app/views/pages/index.html.slim
• app/views/shared/_tag_edit_button.html.slim

---

📝 Walkthrough

Walkthrough

タグ編集UIの表示条件を変更し、管理者ログイン時のみ表示するようにしました。また、キャンセルボタンを非インタラクティブな要素から実際のボタン要素に変更しました。

Changes

Cohort / File(s)	Summary
Tag Edit UI Visibility app/views/pages/index.html.slim	タグ編集ボタンの条件付きレンダリングを追加し、admin_login? が真の場合のみ表示するようにしました。
Cancel Button Element Type app/views/shared/_tag_edit_button.html.slim	キャンセルコントロール要素をdivからbutton要素に変更し、type="button" 属性を明示的に設定しました。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

• TagEditButton / TagEditModal を非React化する #9809: タグ編集UI（同一ビューと共有パーシャル内のキャンセルコントロールおよび管理者表示ロジック）を修正する関連PR。

Suggested reviewers

• okuramasafumi

Poem

🐰✨ タグのボタンが生まれ変わり、
管理者だけが見える魔法も施され、
<button> に変身したキャンセルの子よ、
UIはより正しく、より美しく。
兎は今日も実装の喜びを感じるのだ🎉

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	プルリクエストのタイトルは、CodeRabbitレビューの指摘事項に対する修正であることを明確に示しており、変更内容の主要な目的を適切に要約しています。
Description check	✅ Passed	プルリクエストの説明には、概要と2つの具体的な変更内容が日本語で詳細に記載されており、テンプレートの要件をおおむね満たしています。
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix-tag-edit-coderabbit-review

📝 Coding Plan

• Generate coding plan for human review comments

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[komagata]

/rerun