Skip to content

Conversation

@3w36zj6
Copy link
Member

@3w36zj6 3w36zj6 commented May 22, 2025

現在のドキュメントのバージョンを明示することは以前から要請されており、バージョン表記がページ内に存在しないのは実用上でも不便なので、バージョン表記をヘッダーに追加して常にバージョン情報を閲覧できるようにしました。

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Adds a badge to the site header that displays the current documentation version so it’s always visible.

  • Introduce a VersionBadge component rendering v{version}
  • Render VersionBadge next to SiteTitle in both desktop and mobile headers
Comments suppressed due to low confidence (4)

website/src/components/ui/common/Header.tsx:11

  • Consider adding a test to verify that the VersionBadge renders the correct version value in the header.
const VersionBadge = () => (

website/src/components/ui/common/Header.tsx:12

  • In JSX, use className instead of class to apply CSS classes on React elements.
<span class="inline-flex items-center px-2 py-0.5 rounded-full text-xs font-medium bg-gray-200 text-gray-600 ml-2">

website/src/components/ui/common/Header.tsx:22

  • Replace class with className on this <div> to correctly apply styles in React JSX.
<div class="flex items-center">

website/src/components/ui/common/Header.tsx:71

  • This <div> in the mobile header should use className instead of class for React compatibility.
<div class="flex items-center">

Copy link
Member

@kimushun1101 kimushun1101 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ご対応ありがとうございます。確認いたしました。
image

@3w36zj6 3w36zj6 merged commit 002b1a6 into main May 22, 2025
3 checks passed
@3w36zj6 3w36zj6 deleted the feature/add-version-display-to-website branch May 22, 2025 11:53
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.

3 participants