Skip to content

Commit 0fe1b71

Browse files
bmacombejfversluis
authored andcommitted
Rebase and Clean Up (#8107)
1 parent 578685f commit 0fe1b71

File tree

3 files changed

+33
-9
lines changed

3 files changed

+33
-9
lines changed
Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
using Xamarin.Forms.CustomAttributes;
2+
using Xamarin.Forms.Internals;
3+
4+
namespace Xamarin.Forms.Controls.Issues
5+
{
6+
[Preserve(AllMembers = true)]
7+
[Issue(IssueTracker.Github, 6474, "Border and CornerRadius are not respected in UWP when button is disabled", PlatformAffected.UWP)]
8+
public class Issue6474 : TestContentPage
9+
{
10+
protected override void Init()
11+
{
12+
var stackLayout = new StackLayout() { Margin = 25 };
13+
stackLayout.Children.Add(new Label { Text = "Visually verify that all buttons have a corner radius and that no background color extends beyond the border." });
14+
stackLayout.Children.Add(new Button { CornerRadius = 10, BorderColor = Color.Red, Text = "Enabled" });
15+
stackLayout.Children.Add(new Button { CornerRadius = 10, BorderColor = Color.Red, Text = "Disabled", IsEnabled = false });
16+
Content = stackLayout;
17+
}
18+
}
19+
}

Xamarin.Forms.Controls.Issues/Xamarin.Forms.Controls.Issues.Shared/Xamarin.Forms.Controls.Issues.Shared.projitems

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -20,12 +20,13 @@
2020
<Compile Include="$(MSBuildThisFileDirectory)CollectionViewHeaderFooterView.cs" />
2121
<Compile Include="$(MSBuildThisFileDirectory)CollectionViewItemsUpdatingScrollMode.cs" />
2222
<Compile Include="$(MSBuildThisFileDirectory)Issue3475.cs" />
23+
<Compile Include="$(MSBuildThisFileDirectory)Issue6476.cs" />
2324
<Compile Include="$(MSBuildThisFileDirectory)Issue7825.cs" />
2425
<Compile Include="$(MSBuildThisFileDirectory)Issue5354.xaml.cs">
2526
<SubType>Code</SubType>
26-
</Compile>
27-
<Compile Include="$(MSBuildThisFileDirectory)Issue5868.cs" />
28-
<Compile Include="$(MSBuildThisFileDirectory)Issue6963.cs" />
27+
</Compile>
28+
<Compile Include="$(MSBuildThisFileDirectory)Issue5868.cs" />
29+
<Compile Include="$(MSBuildThisFileDirectory)Issue6963.cs" />
2930
<Compile Include="$(MSBuildThisFileDirectory)Issue7253.cs" />
3031
<Compile Include="$(MSBuildThisFileDirectory)Issue7581.cs" />
3132
<Compile Include="$(MSBuildThisFileDirectory)Issue7621.xaml.cs">
@@ -1591,4 +1592,4 @@
15911592
<Generator>MSBuild:UpdateDesignTimeXaml</Generator>
15921593
</EmbeddedResource>
15931594
</ItemGroup>
1594-
</Project>
1595+
</Project>

Xamarin.Forms.Platform.UAP/FormsButton.cs

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@ public class FormsButton : Windows.UI.Xaml.Controls.Button
1616
new PropertyMetadata(default(Brush), OnBackgroundColorChanged));
1717

1818
WContentPresenter _contentPresenter;
19+
Windows.UI.Xaml.Controls.Grid _rootGrid;
1920

2021
public Brush BackgroundColor
2122
{
@@ -46,6 +47,7 @@ protected override void OnApplyTemplate()
4647
base.OnApplyTemplate();
4748

4849
_contentPresenter = GetTemplateChild("ContentPresenter") as WContentPresenter;
50+
_rootGrid = GetTemplateChild("RootGrid") as Windows.UI.Xaml.Controls.Grid;
4951

5052
UpdateBackgroundColor();
5153
UpdateBorderRadius();
@@ -73,11 +75,13 @@ void UpdateBackgroundColor()
7375

7476
void UpdateBorderRadius()
7577
{
76-
if (_contentPresenter != null)
77-
{
78-
var radius = BorderRadius == -1 ? 0 : BorderRadius;
79-
_contentPresenter.CornerRadius = new Windows.UI.Xaml.CornerRadius(radius);
80-
}
78+
var radius = BorderRadius == -1 ? 0 : BorderRadius;
79+
var cornerRadius = new Windows.UI.Xaml.CornerRadius(radius);
80+
if (_contentPresenter != null)
81+
_contentPresenter.CornerRadius = cornerRadius;
82+
83+
if(_rootGrid != null)
84+
_rootGrid.CornerRadius = cornerRadius;
8185
}
8286

8387
public void UpdateCharacterSpacing(int characterSpacing)

0 commit comments

Comments
 (0)