Skip to content

Conversation

@MrHinsh
Copy link
Member

@MrHinsh MrHinsh commented Jul 30, 2025

This pull request updates the guide-version-select.html file to improve the handling of default values for internationalised strings and ensure consistent formatting in the version dropdown.

Improvements to internationalisation and formatting:

  • Added the default function to provide fallback values for the i18n keys guide_version_latest and guide_version_current, ensuring proper defaults ("latest" and "current") are displayed when translations are unavailable. ([module/layouts/_partials/components/guide/guide-version-select.htmlL15-R38](https://github.com/nkdAgility/HugoGuides/pull/5/files#diff-c404c24e2469819d871a8b512b23fec7c8517a971c91017d8f568f621135d913L15-R38))
  • Replaced inline parentheses with a non-breaking space ( ) before the parentheses for better visual spacing and formatting consistency in the version dropdown and version display. ([module/layouts/_partials/components/guide/guide-version-select.htmlL15-R38](https://github.com/nkdAgility/HugoGuides/pull/5/files#diff-c404c24e2469819d871a8b512b23fec7c8517a971c91017d8f568f621135d913L15-R38))

Copilot AI review requested due to automatic review settings July 30, 2025 08:34
@MrHinsh MrHinsh self-assigned this Jul 30, 2025
@MrHinsh MrHinsh enabled auto-merge July 30, 2025 08:35
Copy link

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 improves the guide version dropdown by adding fallback values for internationalization keys and enhancing visual formatting with better spacing.

  • Added default function to provide fallback values ("latest" and "current") for i18n keys when translations are unavailable
  • Replaced inline parentheses with non-breaking space ( ) before parentheses for improved visual spacing

@MrHinsh MrHinsh added this pull request to the merge queue Jul 30, 2025
Merged via the queue into main with commit 09d82a9 Jul 30, 2025
5 checks passed
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