WindowsCommunityToolkit icon indicating copy to clipboard operation
WindowsCommunityToolkit copied to clipboard

AttachedDropShadow size changes abnormally

Open chenjt2001 opened this issue 2 years ago • 3 comments

Describe the bug

AttachedDropShadow display problem when the size of the element that owns the AttachedDropShadow changes.

` <Page x:Class="Temp.MainPage" xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation" xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml" xmlns:local="using:Temp" xmlns:d="http://schemas.microsoft.com/expression/blend/2008" xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006" mc:Ignorable="d" xmlns:ui="using:Microsoft.Toolkit.Uwp.UI" Background="{ThemeResource ApplicationPageBackgroundThemeBrush}">

<Grid>
    <Grid.ColumnDefinitions>
        <ColumnDefinition Width="1*"/>
        <ColumnDefinition Width="1*"/>
    </Grid.ColumnDefinitions>

    <Border x:Name="ShadowTarget" Grid.Column="0"/>

    <Border Grid.Column="1" Background="AliceBlue">
        <ui:Effects.Shadow>
            <ui:AttachedDropShadow BlurRadius="60"
                               Color="Black"
                               CastTo="{Binding ElementName=ShadowTarget}"/>
        </ui:Effects.Shadow>
    </Border>
</Grid>
`

Regression

No response

Reproducible in sample app?

  • [ ] This bug can be reproduced in the sample app.

Steps to reproduce

1. Run the Application.
2. Change the window size.

Expected behavior

The shadow remains displayed on the entire edge of the Border.

Screenshots

  1. When the application starts: 捕获

  2. After resizing the window: 捕获2 捕获3

Windows Build Number

  • [ ] Windows 10 1809 (Build 17763)
  • [ ] Windows 10 1903 (Build 18362)
  • [ ] Windows 10 1909 (Build 18363)
  • [ ] Windows 10 2004 (Build 19041)
  • [ ] Windows 10 20H2 (Build 19042)
  • [ ] Windows 10 21H1 (Build 19043)
  • [ ] Windows 11 21H2 (Build 22000)
  • [x] Other (specify)

Other Windows Build number

Windows 21H2 (19044.1645)

App minimum and target SDK version

  • [ ] Windows 10, version 1809 (Build 17763)
  • [X] Windows 10, version 1903 (Build 18362)
  • [ ] Windows 10, version 1909 (Build 18363)
  • [X] Windows 10, version 2004 (Build 19041)
  • [ ] Other (specify)

Other SDK version

No response

Visual Studio Version

2022

Visual Studio Build Number

17.1.6

Device form factor

Desktop

Nuget packages

Microsoft.NETCore.UniversalWindowsPlatform: 6.2.13
Microsoft.Toolkit.Uwp.UI: 7.1.2

Additional context

If we set the Border's Background to Transparent, we can find that the size of the window changes in time, but the aspect ratio of the shadow remains unchanged. Maybe that's the reason for the problem.

Help us help you

Yes, but only if others can assist.

chenjt2001 avatar May 05 '22 15:05 chenjt2001

Hello chenjt2001, thank you for opening an issue with us!

I have automatically added a "needs triage" label to help get things started. Our team will analyze and investigate the issue, and escalate it to the relevant team if possible. Other community members may also look into the issue and provide feedback 🙌

ghost avatar May 05 '22 15:05 ghost

Thanks for filing an issue @chenjt2001!

We're registering to the SizeChanged event here, so I would have suspected this to be fired in this scenario to update the shadow:

https://github.com/CommunityToolkit/WindowsCommunityToolkit/blob/00e9790337b17115ca9f7d2b966422f7f221f275/Microsoft.Toolkit.Uwp.UI/Shadows/AttachedDropShadow.cs#L140-L160

Maybe this is related to the note here about the existing rendersize vs. actualsize? Or possibly a delay between those values updating and the next layout pass occurring... Maybe we need to dispatch this or use the CompositionTargetHelper to ensure that the element has the correct size (though I would have thought it should have that by the SizeChanged is fired).

If this is enough for you to dig-in and debug, let me know if you want to take a look at it. If you need more assistance in getting the sample app to run locally to reproduce, let us know too. Thanks!

michael-hawker avatar May 05 '22 18:05 michael-hawker

Thanks. I found that the RenderSize is correct and the problem is that the mask is not updated. See the PR #4549.

chenjt2001 avatar May 06 '22 08:05 chenjt2001

I copy pasted the Grid code in the sample app and see the issue when we adjust the window size changes abnormally and the PR created fixes the issue(Shadow remains constant as we adjust the window size)

LalithaNadimpalli avatar Sep 02 '22 16:09 LalithaNadimpalli