Skip to content

fix: the colon should invisible when ItemsAlignmen=Plain.#884

Merged
zdpcdt merged 2 commits intomainfrom
issue/883
Feb 10, 2026
Merged

fix: the colon should invisible when ItemsAlignmen=Plain.#884
zdpcdt merged 2 commits intomainfrom
issue/883

Conversation

@zdpcdt
Copy link
Collaborator

@zdpcdt zdpcdt commented Feb 10, 2026

No description provided.

@github-actions
Copy link

github-actions bot commented Feb 10, 2026

Test Results

724 tests  +1   724 ✅ +1   15s ⏱️ -1s
  2 suites ±0     0 💤 ±0 
  2 files   ±0     0 ❌ ±0 

Results for commit 14fa9f0. ± Comparison against base commit 4d5d105.

♻️ This comment has been updated with latest results.

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 fixes a UI bug in the Descriptions control where the colon separator was always visible in Plain mode, even when no label was present. The fix adds a style rule that conditionally shows the colon only when the Label property is not null or empty.

Changes:

  • Added conditional visibility logic for the colon in ItemAlignment=Plain mode using StringConverters.IsNotNullOrEmpty

@zdpcdt
Copy link
Collaborator Author

zdpcdt commented Feb 10, 2026

@copilot add HeadlessTest for this PR.
When the ItemAlignment=Plain and the Label is not null or empty, DescriptionItem should show the colon.

Copy link
Contributor

Copilot AI commented Feb 10, 2026

@zdpcdt I've opened a new pull request, #885, to work on those changes. Once the pull request is ready, I'll request review from you.

… mode (#885)

* Initial plan

* test: add HeadlessTest for colon visibility in Plain mode

Co-authored-by: zdpcdt <54255897+zdpcdt@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: zdpcdt <54255897+zdpcdt@users.noreply.github.com>
@zdpcdt zdpcdt requested a review from rabbitism February 10, 2026 15:14
@github-actions
Copy link

Code Coverage

Package Line Rate Branch Rate Complexity Health
Ursa 42% 32% 9051
Ursa.Themes.Semi 62% 68% 1021
Summary 50% (11013 / 22011) 35% (2692 / 7747) 10072

@zdpcdt zdpcdt merged commit 0ad5d3a into main Feb 10, 2026
4 checks passed
@zdpcdt zdpcdt deleted the issue/883 branch February 10, 2026 15:46
zdpcdt added a commit that referenced this pull request Feb 12, 2026
* fix: the colon should invisible when ItemsAlignmen=Plain.

* test: add HeadlessTest for colon visibility in DescriptionsItem Plain mode (#885)

* Initial plan

* test: add HeadlessTest for colon visibility in Plain mode

Co-authored-by: zdpcdt <54255897+zdpcdt@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: zdpcdt <54255897+zdpcdt@users.noreply.github.com>

---------

Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
(cherry picked from commit 0ad5d3a)
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.

DescriptionsItem在Label为空时应该隐藏 TextBlock#PART_Colon

4 participants