support fully rounded corner radius

[Vinci10]

Description:

(Based on #5830)
Added a new constant FullyRoundedCornerRadius for pill/circular shapes as per Material Design.

Checklist:

• Tests included.
• Lint and formatter run with no errors.
• Tests all pass.

Where applicable:

• Public APIs match existing style and have Since: line.
• Any breaking changes have a deprecation path or have been discussed.
• Check for binary size increases when importing new modules.

[andydotxyz]

I don't understand this API sorry. I think it's the same as setting the radius on a rectangle to be half the width or height.
Is that right or is it something different?

[coveralls]

coverage: 62.284% (-0.04%) from 62.327%
when pulling bdeae56 on Vinci10:fully_rounded_corner_radius
into 37cb543 on fyne-io:develop.

[Vinci10]

Right, the API is essentially the same as setting the radius on a rectangle to be half the width or height. Originally, this approach was reserved for circles, but I wanted to extend similar functionality to rectangles as well.
Developers can simply implement this logic on their side, but if the size of the shape changes, the radius would also need to be adjusted accordingly. By handling this directly within the painter, where the size is known before drawing, it simplify the development process and reduce the need for repetitive calculations.

[andydotxyz]

but if the size of the shape changes, the radius would also need to be adjusted accordingly

Thanks, this makes sense as a use-case.

Essentially this could be called RadiusMaximum to avoid having to learn what "fully rounded" means then?

p.s. it would need to be added into the software painter as well - in this case it should be straight forward.

[Vinci10]

but if the size of the shape changes, the radius would also need to be adjusted accordingly

Thanks, this makes sense as a use-case.

Essentially this could be called RadiusMaximum to avoid having to learn what "fully rounded" means then?

p.s. it would need to be added into the software painter as well - in this case it should be straight forward.

Great! I will update the PR, should I also add some unit tests ?

[andydotxyz]

should I also add some unit tests ?

Yes please - always (unless not possible)

[Vinci10]

I added RadiusMaximum to software painter and included tests.

[andydotxyz]

Nice one, thanks for this!