-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Conversation
Need to have a look at the review comments in the original PR |
88868f4
to
303e01d
Compare
83b7031
to
f723d7d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-
What does ButtonSource do? can we add the image to UWP and iOS so we can test it with VSM in all platforms?
-
Padding seems to be applied only on text, not the all control viewport.
-
Can we add a UITests for iOS and Android
-
On UWP when i hover the border goes away.
-
Is there a way to change the color of the Radio (Circle) on UWP ?! seems we can only change the TextColor, but on iOS seems TextColor changes color of text and circle
|
||
SetControlPropertiesFromProxy(); | ||
|
||
_useLegacyColorManagement = e.NewElement.UseLegacyColorManagement(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we have a new Control why we are checking for LegacyColorManagement?
} | ||
); | ||
|
||
var borderRadiusContainer = new ViewContainer<RadioButton>(Test.Button.BorderRadius, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be CornerRadius no ? but it also doesn't work for me if i change to something like:
var borderRadiusContainer = new ViewContainer<RadioButton>(Test.Button.BorderRadius,
new RadioButton
{
Text = "BorderRadius",
BackgroundColor = Color.Transparent,
BorderColor = Color.Red,
BorderWidth = 1,
CornerRadius = 5
}
);
f723d7d
to
53120fd
Compare
namespace Xamarin.Forms | ||
{ | ||
[RenderWith(typeof(_RadioButtonRenderer))] | ||
public class RadioButton : Button, IElementConfiguration<RadioButton> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why don't you implements IFontElement
, ITextElement
, IBorderElement
instead of inheriting Button
directly.
Description of Change
Takeover from the RadioButton PR in #5349. I've copied the relevant parts here.
Issues Resolved
API Changes
Added:
Platforms Affected
Behavioral/Visual Changes
GroupName property
The "RadioButton group Gallery - Legacy" has three tabs which show group behavior.
Radio-buttons with group name null are grouped by their parent.
Radio-buttons with same group name, no matter the hierarchy, are grouped at page level.
Video: https://www.youtube.com/watch?v=fwYSmvjCfbw
Before/After Screenshots
Testing Procedure
See the newly added gallery pages
PR Checklist
TODO