Fix inconsistency between collider constructors and Bevy's primitives

[Jondolf]

Objective

The new constructors for Bevy's Cylinder, Capsule, and Cone shapes take a radius first, and a height second.

Capsule::new(radius, height)

However, our collider constructors use the opposite order:

Collider::capsule(height, radius)

The reason for this is that I originally followed Parry's argument ordering, and Bevy's primitive shapes were added afterwards with a different ordering.

This is a very unfortunate inconsistency and can easily lead to confusion. Even though changing this will lead to very clear breakage for a large number of applications, I believe it is important that we align ourselves with Bevy here.

Solution

Make the argument ordering for all collider constructors match Bevy.

---

Migration Guide

To match Bevy's Cylinder, Capsule, and Cone, the order of arguments has changed for some Collider constructors.

• Use Collider::cylinder(radius, height) instead of Collider::cylinder(height, radius).
• Use Collider::capsule(radius, height) instead of Collider::capsule(height, radius).
• Use Collider::capsule_endpoints(radius, a, b) instead of Collider::capsule_endpoints(a, b, radius).
• Use Collider::cone(radius, height) instead of Collider::cone(height, radius).