Skip to content

Fixed elevation border brush for Light and Dark theme #10567

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

dipeshmsft
Copy link
Member

@dipeshmsft dipeshmsft commented Mar 14, 2025

Fixes #10569

Description

Since #10502 is fixed now, I have raised this PR to fix the elevation brushes for Light and Dark themes. Elevation brushes use RelativeTransform alongwith an Absolute mode LinearGradientBrush.

Customer Impact

Fixes Fluent theme resources

Regression

No

Testing

Local testing

Risk

Minimal

Microsoft Reviewers: Open in CodeFlow

@Copilot Copilot AI review requested due to automatic review settings March 14, 2025 05:54
@dipeshmsft dipeshmsft requested review from a team as code owners March 14, 2025 05:54
Copy link
Contributor

@Copilot 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.

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

@dotnet-policy-service dotnet-policy-service bot added the PR metadata: Label to tag PRs, to facilitate with triage label Mar 14, 2025
@dipeshmsft dipeshmsft added this to the .NET 10 milestone Mar 14, 2025
Copy link

codecov bot commented Mar 14, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 11.41184%. Comparing base (a354bec) to head (7ba61f1).
Report is 2 commits behind head on main.

Additional details and impacted files
@@                 Coverage Diff                 @@
##                main      #10567         +/-   ##
===================================================
- Coverage   11.54183%   11.41184%   -0.12999%     
===================================================
  Files           3214        3214                 
  Lines         648502      648502                 
  Branches       71510       71510                 
===================================================
- Hits           74849       74006        -843     
- Misses        572401      573335        +934     
+ Partials        1252        1161         -91     
Flag Coverage Δ
Debug 11.41184% <ø> (-0.12999%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@siagupta0202 siagupta0202 self-assigned this Mar 16, 2025
@siagupta0202 siagupta0202 merged commit e3ce5f8 into dotnet:main Mar 16, 2025
8 checks passed
@dipeshmsft dipeshmsft deleted the fluent/misc/revert-elevation-brush-workaround branch March 31, 2025 10:25
@dipeshmsft dipeshmsft modified the milestones: .NET 10, 10.0.0 Apr 9, 2025
@github-actions github-actions bot locked and limited conversation to collaborators May 9, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
PR metadata: Label to tag PRs, to facilitate with triage Win 11 Theming
Projects
Status: ☑ Done
Development

Successfully merging this pull request may close these issues.

[ Fluent ] Elevation brushes for Light / Dark theme use Relative mode instead of Absolute and RelativeTransform.
3 participants