Xamarin.forms: [Bug] [Shapes 4.8-pre2] Justice for baby elephant!

[davidbritch]

Description

I've just updated my Shapes demos sample from 4.7 to 4.8-pre2, in order to try Shapes with Brushes. As a result, Shapes that don't set the Stroke are now defaulting to a black stroke. This seems to be a result of changing the Stroke and Fill properties to be of type Brush, which defaults to Color.Default? To me, it seems like a better user experience is for users to opt into a Stroke colour, rather than having to opt out of the black outline.

Steps to Reproduce

1. Run the attached sample
2. Browse to the baby elephant page.

Expected Behavior

Baby elephant looks correct (no black stroke).

Actual Behavior

Baby elephant looks incorrect due to black stroke on each Path object.

Basic Information

• Version with issue: 4.8-pre2
• Last known good version: 4.7
• IDE: VSMac 8.6.7
• Platform Target Frameworks:
  
  
  
  • iOS: 13.6
  
  
  • Android: API 28
  
  

Screenshots

Baby elephant in 4.7:

Baby elephant in 4.8-pre2:

Reproduction Link

ShapesDemos.zip

[StephaneDelcroix]

you won the 'best issue title' award