Popover.md: Update docs for IsAnimated

RadekVyM / SimpleToolkit

SimpleToolkit is a .NET MAUI library of helpers and simple, fully customizable controls, such as SimpleShell - custom Shell implementation that allows you to create unique navigation experiences.

MIT License

434 stars 44 forks source link

Popover.md: Update docs for IsAnimated #57

Closed breyed closed 3 months ago

breyed commented 3 months ago

Update documentation based on discussion in #51.

Indicate default.
Provide detail on Android behavior.
Remove descriptive text redundant with property name.

RadekVyM commented 3 months ago

Hi,

the description of the Android behavior does not align with the behavior in the 5.1.2 update. More info in a comment in #51. Also, I would like to keep the information that IsAnimated is a boolean property.

breyed commented 3 months ago

Yes, I posted the PR for the docs before I learned about the update method, so the PR quickly became out of date.

Regarding, the explicit mention of the property type, since the Is in the naming of IsAnimated makes the type obvious, there’s no practical loss of information to use the more concise verbiage that omits the type. That said, it’s a stylist matter, so totally your call.