Skip to content

Mark truncated text with an ellipsis app-wide - #5341

Merged
Gabriel Dufresne (GabrielDuf) merged 1 commit into
mainfrom
fix/5336-truncated-text-ellipsis
Sep 1, 2026
Merged

Mark truncated text with an ellipsis app-wide#5341
Gabriel Dufresne (GabrielDuf) merged 1 commit into
mainfrom
fix/5336-truncated-text-ellipsis

Conversation

@GabrielDuf

Copy link
Copy Markdown
Contributor

This pull request refactors several UI components to improve layout consistency and text handling across the application. The main changes involve replacing StackPanel layouts with Grid layouts for better alignment of icons and text, and setting a global text trimming style to ensure long text is properly ellipsized. These updates enhance the visual appearance and usability, especially when dealing with variable-length text.

UI Layout Refactoring:

  • Replaced multiple instances of StackPanel with Grid layouts in files such as ManageShortcutsWindow.axaml, ManageAutoUpdatesWindow.axaml, OperationHistoryPage.axaml, SettingsBasePage.axaml, and SidebarView.axaml. This change ensures better alignment and spacing between icons and text, especially for items with variable-length content.

Text Handling Improvements:

  • Added a global style for TextBlock in Styles.Common.axaml to set TextTrimming to CharacterEllipsis, ensuring that overflowing text is consistently truncated with an ellipsis throughout the application.

These changes collectively improve the app's look and feel, making UI elements more robust and visually consistent.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Refines Avalonia layouts so constrained text can consistently display ellipses.

Changes:

  • Replaces horizontal stacks with constrained icon/text grids.
  • Applies global character-ellipsis trimming to TextBlock.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
AbstractPackagesPage.axaml Constrains package and source labels.
SidebarView.axaml Constrains navigation labels.
SettingsBasePage.axaml Improves restart-banner layout.
OperationHistoryPage.axaml Constrains operation labels.
ManageShortcutsWindow.axaml Constrains shortcut fields.
ManageAutoUpdatesWindow.axaml Improves warning layout.
Styles.Common.axaml Enables global ellipsis trimming.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@randy-but-a-ro randy-but-a-ro Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🤖 Pull request was approved automatically: the AI review is complete and all its review threads are resolved. 🎉

Integration Details
{
	"deliveryId": "842f84b0-a605-11f1-9130-4694db736c89",
	"headSha": "cdd8849ac9d77da1376cc8577889a35522d981f8",
	"reviewer": "copilot-pull-request-reviewer[bot]"
}

@GabrielDuf
Gabriel Dufresne (GabrielDuf) merged commit 1cc3f46 into main Sep 1, 2026
2 checks passed
@GabrielDuf
Gabriel Dufresne (GabrielDuf) deleted the fix/5336-truncated-text-ellipsis branch September 1, 2026 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Truncated text in main grid view is not marked as such

2 participants