Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

[Bug][Shapes] RoundRectangleGeometry clipping error #12285

Closed
davidbritch opened this issue Sep 28, 2020 · 0 comments · Fixed by #12286
Closed

[Bug][Shapes] RoundRectangleGeometry clipping error #12285

davidbritch opened this issue Sep 28, 2020 · 0 comments · Fixed by #12286
Assignees
Labels
a/shapes e/2 🕑 2 in-progress This issue has an associated pull request that may resolve it! t/bug 🐛
Milestone

Comments

@davidbritch
Copy link

Description

The RoundRectangleGeometry class has been added to Forms 5.0. However, its not producing the result I'd expect when using it for image clipping:

            <Label Text="Clipped image with RectangleGeometry" />
            <Image Source="monkeyface.png">
                <Image.Clip>
                    <RectangleGeometry Rect="40,100,300,150" />
                </Image.Clip>
            </Image>
            <Label Text="Clipped image with RoundRectangleGeometry" />
            <Image Source="monkeyface.png">
                <Image.Clip>
                    <RoundRectangleGeometry CornerRadius="5"
                                            Rect="40,100,300,150" />
                </Image.Clip>
            </Image>

Note that the two geometries use identical rectangles, with the RoundRectangleGeometry adding a corner radius. I'd therefore expect the two resulting images to be similar. But they aren't:

Simulator Screen Shot - iPhone 11 - 2020-09-28 at 12 34 45

Steps to Reproduce

  1. Run the attached sample.
  2. Browse to the "Clipping demos" page.

Expected Behavior

Image clipped with RoundRectangleGeometry is identical to RectangleGeometry clipping, but with rounded corners.

Actual Behavior

Image clipped with RoundRectangleGeometry is not clipped correctly.

Basic Information

  • Version with issue: 5.0-pre1
  • IDE: VSMac 8.7.8
  • Platform Target Frameworks:
    • iOS: 14.0
    • Android: API 29
    • UWP: Not tried but will be a problem.

Reproduction Link

ShapesDemos.zip

@davidbritch davidbritch added t/bug 🐛 s/unverified New report that has yet to be verified labels Sep 28, 2020
@jsuarezruiz jsuarezruiz self-assigned this Sep 28, 2020
@jsuarezruiz jsuarezruiz added a/shapes e/2 🕑 2 and removed s/unverified New report that has yet to be verified labels Sep 28, 2020
@samhouts samhouts added this to New in Triage Sep 28, 2020
@samhouts samhouts added this to the 5.0.0 milestone Sep 29, 2020
@samhouts samhouts moved this from New to Ready For Work in Triage Sep 30, 2020
@samhouts samhouts added the in-progress This issue has an associated pull request that may resolve it! label Oct 1, 2020
@samhouts samhouts added this to In Progress in vNext+1 (5.0.0) Oct 1, 2020
Triage automation moved this from Ready For Work to Closed Oct 2, 2020
vNext+1 (5.0.0) automation moved this from In Progress to Done Oct 2, 2020
@samhouts samhouts removed this from Closed in Triage Nov 3, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a/shapes e/2 🕑 2 in-progress This issue has an associated pull request that may resolve it! t/bug 🐛
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

3 participants