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

[ShadowContainer] Cache issue on CornerRadius #782

Closed
1 of 11 tasks
rafael-rosa-knowcode opened this issue Aug 30, 2023 · 1 comment · Fixed by #788, #848 or #849
Closed
1 of 11 tasks

[ShadowContainer] Cache issue on CornerRadius #782

rafael-rosa-knowcode opened this issue Aug 30, 2023 · 1 comment · Fixed by #788, #848 or #849
Assignees
Labels
control/shadows kind/enhancement New feature or request. triage/untriaged Indicates an issue requires triaging or verification.

Comments

@rafael-rosa-knowcode
Copy link
Contributor

rafael-rosa-knowcode commented Aug 30, 2023

What would you like to be added:

When running the tests for the ShadowsContainer.
We use the a
Border (Width="200", Height="200"),
image

Rectangle (Width="200", Height="200") and
image

Border (Width="200", Height="200", CornerRadius="0,200,0,200"),
image

When the test is performed with the corner radius parameters in the border. Shadows assumes the element's corner radius.
However, when we go back to the border element without the corner radius, it is being cut to the shape of the one with the corner radius.

This is causing the test that has the same parameters to fail during execution.

When using 9 and -9.
image

When using 10 and -10
image

The situation only happens when we run the tests at once. Individually tests pass.

Why is this needed:

It could be a problem when creating the test, but it could be a problem with the cornerRadius cache.

For which Platform:

  • WebAssembly
  • Android
  • iOS
  • macOS (AppKit)
  • Mac Catalyst
  • Skia
    • WPF
    • GTK (Linux)
    • Linux Framebuffer
    • Tizen
  • Windows

Anything else we need to know?

@agneszitte
Copy link
Contributor

Re-Open for backports

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment