Skip to content
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

Added TextControlElevationFocusedBorderBrush to fluent resources #10685

Conversation

dipeshmsft
Copy link
Member

@dipeshmsft dipeshmsft commented Mar 31, 2025

Description

This brush should be used as the border in all the text controls when they are focused. However, due to the issue in LinearGradientBrush we were using different workarounds to achieve the same effect in TextBox and related controls wrt to style.

Customer Impact

Will allow fixing fluent style for TextBox and other related controls.

Regression

NA

Testing

Local app testing

Risk

None, new brushes added.

Microsoft Reviewers: Open in CodeFlow

@Copilot Copilot bot review requested due to automatic review settings March 31, 2025 13:37
@dipeshmsft dipeshmsft requested review from a team as code owners March 31, 2025 13:37
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 wasn't able to review any files in this pull request.

Files not reviewed (6)
  • src/Microsoft.DotNet.Wpf/src/Themes/PresentationFramework.Fluent/Resources/Theme/Dark.xaml: Language not supported
  • src/Microsoft.DotNet.Wpf/src/Themes/PresentationFramework.Fluent/Resources/Theme/HC.xaml: Language not supported
  • src/Microsoft.DotNet.Wpf/src/Themes/PresentationFramework.Fluent/Resources/Theme/Light.xaml: Language not supported
  • src/Microsoft.DotNet.Wpf/src/Themes/PresentationFramework.Fluent/Themes/Fluent.Dark.xaml: Language not supported
  • src/Microsoft.DotNet.Wpf/src/Themes/PresentationFramework.Fluent/Themes/Fluent.HC.xaml: Language not supported
  • src/Microsoft.DotNet.Wpf/src/Themes/PresentationFramework.Fluent/Themes/Fluent.Light.xaml: Language not supported

@dipeshmsft dipeshmsft requested a review from siagupta0202 March 31, 2025 13:37
@dotnet-policy-service dotnet-policy-service bot added the PR metadata: Label to tag PRs, to facilitate with triage label Mar 31, 2025
Copy link

codecov bot commented Mar 31, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 11.26422%. Comparing base (4aff730) to head (881e7f6).
Report is 1 commits behind head on main.

Additional details and impacted files
@@                 Coverage Diff                 @@
##                main      #10685         +/-   ##
===================================================
- Coverage   11.28308%   11.26422%   -0.01887%     
===================================================
  Files           3353        3353                 
  Lines         668062      668062                 
  Branches       74980       74980                 
===================================================
- Hits           75378       75252        -126     
- Misses        591433      591564        +131     
+ Partials        1251        1246          -5     
Flag Coverage Δ
Debug 11.26422% <ø> (-0.01887%) ⬇️

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 31, 2025
@siagupta0202 siagupta0202 merged commit 5f73690 into dotnet:main Mar 31, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR metadata: Label to tag PRs, to facilitate with triage Win 11 Theming
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants