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

[Android, iOS] Fixed issue rendering Rectangle using Radius #11418

Merged
merged 9 commits into from Aug 17, 2020
Merged

Conversation

jsuarezruiz
Copy link
Contributor

Description of Change

Fixed issue rendering Rectangle using RadiusX and RadiusY. The problem was assigning a radius before setting the size of the rectangle.

Issues Resolved

API Changes

None

Platforms Affected

  • Core/XAML (all platforms)
  • iOS
  • Android

Behavioral/Visual Changes

None

Before/After Screenshots

fix11413

Testing Procedure

Launch Core Gallery and navigate to the issue 11413. If a black square with corner radius is rendered inside a red rectangle, the test has passed.

PR Checklist

  • Targets the correct branch
  • Tests are passing (or failures are unrelated)

@samhouts samhouts removed the request for review from PureWeen July 13, 2020 22:27
@samhouts samhouts assigned samhouts and unassigned PureWeen Jul 13, 2020
@samhouts samhouts added the i/high Completely doesn't work, crashes, or is unusably slow, has no obvious workaround; occurs less often label Jul 13, 2020
@samhouts samhouts changed the base branch from 4.7.0 to 4.8.0 July 29, 2020 23:32
@samhouts
Copy link
Member

@jsuarezruiz Please fix the conflicts! Thanks!

@samhouts samhouts removed this from In Review in 4.7.0 Jul 30, 2020
@samhouts samhouts added this to In Review in vCurrent (4.8.0) Jul 30, 2020
@samhouts samhouts changed the base branch from 4.8.0 to 5.0.0 August 4, 2020 22:08
@samhouts samhouts added this to the 5.0.0 milestone Aug 4, 2020
@samhouts samhouts added the p/UWP label Aug 4, 2020
@samhouts samhouts removed this from In Review in vCurrent (4.8.0) Aug 4, 2020
@samhouts samhouts added this to In Review in vNext+1 (5.0.0) Aug 11, 2020
@samhouts samhouts removed the request for review from hartez August 13, 2020 21:39
@samhouts samhouts assigned rmarinho and unassigned hartez Aug 13, 2020
@samhouts samhouts added the approved Has two approvals, no pending reviews, and no changes requested label Aug 14, 2020
@samhouts samhouts merged commit 9a9b842 into 5.0.0 Aug 17, 2020
vNext+1 (5.0.0) automation moved this from In Review to Done Aug 17, 2020
@samhouts samhouts deleted the fix-11413 branch August 17, 2020 17:40
@samhouts samhouts added this to Done in Sprint 175 Aug 17, 2020
myroot pushed a commit to myroot/Xamarin.Forms that referenced this pull request Aug 19, 2020
…11418)

* Added repro sample

* Fixed issue on Android

* Fixed the issue on iOS

* Fixed build error

* [iOS] Fix rebase

Co-authored-by: Samantha Houts <samhouts@users.noreply.github.com>
Co-authored-by: Rui Marinho <me@ruimarinho.net>

fixes xamarin#11413
sung-su pushed a commit to sung-su/Xamarin.Forms that referenced this pull request Aug 20, 2020
…11418)

* Added repro sample

* Fixed issue on Android

* Fixed the issue on iOS

* Fixed build error

* [iOS] Fix rebase

Co-authored-by: Samantha Houts <samhouts@users.noreply.github.com>
Co-authored-by: Rui Marinho <me@ruimarinho.net>

fixes xamarin#11413
sung-su pushed a commit to sung-su/Xamarin.Forms that referenced this pull request Aug 20, 2020
…11418)

* Added repro sample

* Fixed issue on Android

* Fixed the issue on iOS

* Fixed build error

* [iOS] Fix rebase

Co-authored-by: Samantha Houts <samhouts@users.noreply.github.com>
Co-authored-by: Rui Marinho <me@ruimarinho.net>

fixes xamarin#11413
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a/shapes approved Has two approvals, no pending reviews, and no changes requested i/high Completely doesn't work, crashes, or is unusably slow, has no obvious workaround; occurs less often p/Android p/iOS 🍎 p/UWP t/bug 🐛
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

5 participants