Xamarin.forms: Polygon Shape doesn't work with DynamicResource color binding

[bdgza]

Description

Updating to XF 4.8 from 4.7 the DynamicResource fill color binding on the Polygon shape is no longer recognized. It still works with a fixed color.

Steps to Reproduce

1. Bind a DynamicResource color to the fill color on a Polygon shape in XAML.

Expected Behavior

The DynamicResource color binding is used to fill the Polygon.

Actual Behavior

The Polygon is not drawn at all.

Basic Information

• Version with issue: Xamarin Forms 4.8.0.1269
• Last known good version: 4.7.0.1260
• IDE: Visual Studio for Mac 8.7
• Platform Target Frameworks:
  
  
  
  • iOS: 13.6 (13.20.2.2)
  
  
  • Android: 10 (10.3.1.4)
  
  
• Affected Devices: iOS Simulator, Samsung SM-T395

Screenshots

4.7:

4.8:

Reproduction Link

PolygonDynamicColor.zip

[PureWeen]

@StephaneDelcroix or @jsuarezruiz ?

[jsuarezruiz]

The type supported by the Shapes Fill and Stroke properties in Xamarin.Forms 4.8 is a Brush (SolidColorBrush, LinearGradientBrush, or RadialGradientBrush) instead of a Color.
The first case works because there is a TypeConverter, but in the second, using a Color type, it is not working. Changing the Color resource by a SolidColorBrush fixes the problem.

[StephaneDelcroix]

@jsuarezruiz so you'd like TypeConverters to be applied with DynamicResource, or any kind of Binding too ?

[bdgza]

I define a SolidColorBrush entry in my App.xaml and reference this Brush in my Polygon Fill instead of the Color. But then setting the Color for my SolidColorBrush has the same problem as setting the Color directly.

I can only use a fixed color or StaticResource Color, as soon as I change the Brush Color to a DynamicResource Color it doesn't render the Polygon.

[jsuarezruiz]

Could you take a look to this modified sample:
PolygonDynamicColor.zip
?

[bdgza]

I took a look, but what if I wanted to do it like how it is in this modified modified sample:
PolygonDynamicColor.zip

[jsuarezruiz]

@bdgza Thanks for the new modified sample. Now I see the issue!. Will answer here when I have the fix.

[BurkusCat]

I think this needs reopened @rmarinho. I've posted a comment on #12836 describing more but if the color dynamically changes the polygon shape does not change color. It is behaving like a StaticResource instead of a DynamicResource.

[jsuarezruiz]

@BurkusCat Thanks for the feedback, I will review the issue changing the DynamicResource.