Skip to content

Comments

RTL fixes#908

Open
yosilevy wants to merge 3 commits intohacs:mainfrom
yosilevy:RTL-fixes
Open

RTL fixes#908
yosilevy wants to merge 3 commits intohacs:mainfrom
yosilevy:RTL-fixes

Conversation

@yosilevy
Copy link

No description provided.

Copilot AI review requested due to automatic review settings February 11, 2026 15:40
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 pull request adds Right-to-Left (RTL) text direction support to HACS (Home Assistant Community Store). The changes introduce RTL direction computation and apply it globally, while also providing mechanisms to force LTR (Left-to-Right) display for specific technical content.

Changes:

  • Added global RTL direction detection and application to the document root in the main HacsElement
  • Introduced forceLTR parameter for dialogs to handle technical content (version numbers, URLs, GitHub usernames) that should always display left-to-right
  • Applied code style improvements with consistent trailing commas

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
src/hacs.ts Imports RTL utilities, applies direction globally in _initializeLocalize, adds trailing commas for consistency
src/dashboards/hacs-repository-dashboard.ts Forces LTR direction for repository content (versions, usernames, stats)
src/dashboards/hacs-dashboard.ts Adds forceLTR parameter to "About HACS" dialog
src/components/dialogs/show-hacs-dialog.ts Defines forceLTR as required parameter in HacsFormDialogParams interface, adds trailing commas
src/components/dialogs/hacs-form-dialog.ts Implements forceLTR support with CSS class, adds state property and styling

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

yosilevy and others added 2 commits February 11, 2026 18:27
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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.

1 participant