Skip to content

Conversation

@zhangxuhe1
Copy link
Contributor

Add application version number display next to 'About' menu items in both

mobile and desktop user interfaces. Version is extracted from package.json

and exposed via NEXT_PUBLIC_APP_VERSION environment variable.

Changes:

  • Extract version from package.json in next.config.ts

  • Display version number in mobile user bottom sheet about menu

  • Display version number in desktop user avatar dropdown about menu

  • Maintain consistent styling with existing theme colors

  • Version appears as 'v{version}' with subtle opacity styling

Important

  1. Read CONTRIBUTING.md
  2. Ensure issue exists and you're assigned
  3. Link correctly: Fixes #<issue number>

What & Why

What: Brief description
Why: Problem solved

Fixes #(issue number)

Pre-PR Checklist

Run these:

  • pnpm type-check
  • pnpm format:check
  • pnpm lint
  • pnpm build
  • pnpm i18n:check (if applicable)

Type

  • 🐛 Bug fix
  • ✨ Feature
  • 💥 Breaking change
  • 📚 Docs
  • ♻️ Refactor
  • ⚡ Performance

Screenshots (if UI changes)

Add application version number display next to 'About' menu items in both

mobile and desktop user interfaces. Version is extracted from package.json

and exposed via NEXT_PUBLIC_APP_VERSION environment variable.

Changes:

- Extract version from package.json in next.config.ts

- Display version number in mobile user bottom sheet about menu

- Display version number in desktop user avatar dropdown about menu

- Maintain consistent styling with existing theme colors

- Version appears as 'v{version}' with subtle opacity styling
@zhangxuhe1 zhangxuhe1 requested a review from lyzno1 August 9, 2025 14:42
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. area:ui UI components, layouts, styling, accessibility. type:feature Request for a brand-new capability. labels Aug 9, 2025
- Create dedicated VersionTag component with tag-style design
- Use font-mono for better version number readability
- Support multiple variants and theme-aware styling
- Replace inline version display with reusable component
- Clean up unnecessary comments across affected files
- Position version tags with proper spacing using ml-auto

Components updated:
- mobile/ui/user-bottom-sheet.tsx: Use VersionTag component
- nav-bar/desktop-user-avatar.tsx: Use VersionTag component
- ui/version-tag.tsx: New reusable component
- ui/index.ts: Export VersionTag component
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Aug 9, 2025
@lyzno1 lyzno1 requested a review from Copilot August 9, 2025 15:01
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

This PR adds version display functionality to both mobile and desktop user interface about menu items. The version number is extracted from package.json and displayed alongside the 'About' menu option with consistent styling across platforms.

  • Extracts app version from package.json via NEXT_PUBLIC_APP_VERSION environment variable
  • Creates reusable VersionTag component with multiple styling variants
  • Integrates version display in both mobile bottom sheet and desktop avatar dropdown menus

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
next.config.ts Adds version extraction from package.json and environment variable configuration
components/ui/version-tag.tsx Creates new reusable component for displaying version with multiple styling variants
components/ui/index.ts Exports the new VersionTag component
components/nav-bar/desktop-user-avatar.tsx Integrates version display in desktop user dropdown menu
components/mobile/ui/user-bottom-sheet.tsx Integrates version display in mobile user bottom sheet

@dosubot dosubot bot added the lgtm Looks good to me; approved by a reviewer. label Aug 9, 2025
@lyzno1 lyzno1 merged commit a267274 into main Aug 9, 2025
16 checks passed
@zhangxuhe1 zhangxuhe1 deleted the feat/ui-add-version-display-in-about-menu branch August 10, 2025 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:ui UI components, layouts, styling, accessibility. lgtm Looks good to me; approved by a reviewer. size:L This PR changes 100-499 lines, ignoring generated files. type:feature Request for a brand-new capability.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants