From c1f9722a97f08a2d5b97c266fecd9bde92c728ad Mon Sep 17 00:00:00 2001 From: Joona Aalto Date: Thu, 6 Feb 2025 01:25:40 +0200 Subject: [PATCH] Fix the default layers used by `ColliderConstructorHierarchy` (#649) # Objective #476 changed `CollisionLayers` to use one membership (the first layer) and all filters by default. However, `ColliderConstructorHierarchy` is still using the old default! ## Solution Use `CollisionLayers::default()` (one membership, all filters) by default for `ColliderConstructorHierarchy`. --- ## Migration Guide `ColliderConstructorHierarchy` now defaults to one membership (the first layer) and all filters for the `CollisionLayers` of generated colliders. This is consistent with how `CollisionLayers` already works normally. --- src/collision/collider/constructor.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/collision/collider/constructor.rs b/src/collision/collider/constructor.rs index 76fd36f9..963c279e 100644 --- a/src/collision/collider/constructor.rs +++ b/src/collision/collider/constructor.rs @@ -115,7 +115,7 @@ pub struct ColliderConstructorHierarchy { pub default_constructor: Option, /// The default [`CollisionLayers`] used for colliders in the hierarchy. /// - /// [`CollisionLayers::ALL`] by default. + /// [`CollisionLayers::default()`] by default, with the first layer and all filters. pub default_layers: CollisionLayers, /// The default [`ColliderDensity`] used for colliders in the hierarchy. /// @@ -141,7 +141,7 @@ impl ColliderConstructorHierarchy { pub fn new(default_constructor: impl Into>) -> Self { Self { default_constructor: default_constructor.into(), - default_layers: CollisionLayers::ALL, + default_layers: CollisionLayers::default(), default_density: ColliderDensity(1.0), config: default(), }