Skip to content

Conversation

@jiripolasek
Copy link
Collaborator

@jiripolasek jiripolasek commented Jan 6, 2026

Summary of the Pull Request

This PR improves visuals and a11y of sections:

  • Changes section title color from disabled to secondary text fill.
  • Removes separator line if the text is present and indents
  • Changes gallery grid subtitle color from tertiary to secondary text fill.

Pictures? Pictures!

image image image image

PR Checklist

  • Closes: #xxx
  • Communication: I've discussed this with core contributors already. If the work hasn't been agreed, this work might be rejected
  • Tests: Added/updated and all pass
  • Localization: All end-user-facing strings can be localized
  • Dev docs: Added/updated
  • New binaries: Added on the required places
  • Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

- Changes section title color from disabled to secondary text fill.
- Moves the separator line slightly lower to align with the section title crossbars.
- Changes gallery grid subtitle color from tertiary to secondary text fill.
@jiripolasek jiripolasek self-assigned this Jan 6, 2026
@jiripolasek jiripolasek added the Product-Command Palette Refers to the Command Palette utility label Jan 6, 2026
@Jay-o-Way
Copy link
Collaborator

Just curious - see a few places where a (literal) value is used, where a style would give the same result. Is there a reason to prefer one over the other?

E.G. fontsize 14 = BodyTextBlockStyle; L383 CornerRadius="4" etc.

@zadjii-msft
Copy link
Member

XAML Styler got you on this one 🙃

@zadjii-msft zadjii-msft added the Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something label Jan 8, 2026
@jiripolasek jiripolasek requested a review from niels9001 January 8, 2026 18:05
@jiripolasek
Copy link
Collaborator Author

jiripolasek commented Jan 8, 2026

Screenshots updated to match the changes.

XAML Styler got you on this one 🙃

if only once...

squirrelplaysdead-squirrel-plays-dead

@microsoft-github-policy-service microsoft-github-policy-service bot added Needs-Triage For issues raised to be triaged and prioritized by internal Microsoft teams Needs-Team-Response An issue author responded so the team needs to follow up and removed Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something labels Jan 8, 2026
@niels9001 niels9001 added In for .97 and removed Needs-Triage For issues raised to be triaged and prioritized by internal Microsoft teams labels Jan 8, 2026
@jiripolasek jiripolasek requested a review from niels9001 January 8, 2026 19:29
@jiripolasek jiripolasek removed the Needs-Team-Response An issue author responded so the team needs to follow up label Jan 8, 2026
@niels9001 niels9001 merged commit d48338b into microsoft:main Jan 8, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

In for .97 Product-Command Palette Refers to the Command Palette utility

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants