Skip to content

Conversation

@JeremyKuhne
Copy link
Member

@JeremyKuhne JeremyKuhne commented Aug 25, 2020

Stock objects have a different define in metafile records. Correct values for API usage.

Add rectangle validation to the EMF validator and add regression tests for button and the Interop definitions.

Ported from #3774 / 6f4b818

Customer Impact

  • Flat style buttons and other controls that use a null/hollow brush don't render correctly.

Regression?

  • Yes

Risk

  • Low
Microsoft Reviewers: Open in CodeFlow

Stock objects have a different define in metafile records. Correct values for API usage.

Add rectangle validation to the EMF validator and add regression tests for button and the Interop definitions.
@JeremyKuhne JeremyKuhne requested a review from a team as a code owner August 25, 2020 23:33
@ghost ghost assigned JeremyKuhne Aug 25, 2020
@Pilchie Pilchie added the servicing-approved .NET Shiproom approved the PR for merge label Aug 25, 2020
@Pilchie
Copy link
Member

Pilchie commented Aug 25, 2020

Approved for RC1.

@codecov
Copy link

codecov bot commented Aug 26, 2020

Codecov Report

Merging #3776 into release/5.0-rc1 will decrease coverage by 31.20960%.
The diff coverage is 16.66667%.

@@                    Coverage Diff                     @@
##           release/5.0-rc1       #3776          +/-   ##
==========================================================
- Coverage         67.39766%   36.18806%   -31.20961%     
==========================================================
  Files                 1393         916         -477     
  Lines               504657      250403      -254254     
  Branches             40895       36662        -4233     
==========================================================
- Hits                340127       90616      -249511     
+ Misses              158702      154671        -4031     
+ Partials              5828        5116         -712     
Flag Coverage Δ
#Debug 36.18806% <16.66667%> (-31.20961%) ⬇️
#production 36.18806% <16.66667%> (+0.00198%) ⬆️
#test ?

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

@RussKie RussKie added this to the 5.0 RC1 milestone Aug 26, 2020
@RussKie RussKie merged commit aac5c1b into dotnet:release/5.0-rc1 Aug 26, 2020
@ghost ghost locked as resolved and limited conversation to collaborators Jan 31, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

servicing-approved .NET Shiproom approved the PR for merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants