Removing hit_test from Space

Refs #201

CHANGELOG.md

@@ -79,6 +79,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
   - `Checkable::to_checkbox`
   - `Dynamic::new_radio`
   - `Dynamic::new_checkbox`
+- `Space` no longer implements hit_test. If you need an area to intercept mouse
+  events, wrap the `Space` in a `Custom` widget:
+
+  ```rust
+    Custom::new(Space::colored(Color::RED)).on_hit_test(|_, _| true)
+  ```
 
 ### Changed
 

src/widgets/layers.rs

@@ -12,7 +12,7 @@ use figures::{IntoSigned, IntoUnsigned, Point, Rect, Size, Zero};
 use intentional::Assert;
 
 use super::super::widget::MountedWidget;
-use super::Space;
+use super::{Custom, Space};
 use crate::animation::{AnimationHandle, AnimationTarget, IntoAnimate, Spawn, ZeroToOne};
 use crate::context::{AsEventContext, EventContext, GraphicsContext, LayoutContext, Trackable};
 use crate::styles::components::{EasingIn, ScrimColor};
@@ -1011,7 +1011,9 @@ impl WrapperWidget for ModalLayer {
 
         for to_present in modal.iter().skip(self.presented.len()) {
             self.focus_top_layer = true;
-            layer_widgets.push(Space::colored(context.get(&ScrimColor)));
+            layer_widgets.push(
+                Custom::new(Space::colored(context.get(&ScrimColor))).on_hit_test(|_, _| true),
+            );
             self.presented.push(to_present.clone());
             layer_widgets.push(to_present.clone().centered());
         }

src/widgets/space.rs

@@ -4,7 +4,7 @@ use kludgine::Color;
 
 use crate::context::{GraphicsContext, LayoutContext};
 use crate::styles::components::PrimaryColor;
-use crate::styles::{Component, DynamicComponent, IntoDynamicComponentValue};
+use crate::styles::{DynamicComponent, IntoDynamicComponentValue};
 use crate::value::{IntoValue, Value};
 use crate::widget::Widget;
 use crate::ConstraintLimit;
@@ -76,24 +76,6 @@ impl Widget for Space {
     ) -> Size<UPx> {
         available_space.map(ConstraintLimit::min)
     }
-
-    fn hit_test(
-        &mut self,
-        _location: figures::Point<figures::units::Px>,
-        context: &mut crate::context::EventContext<'_>,
-    ) -> bool {
-        let color = match self.color.get() {
-            ColorSource::Color(color) => color,
-            ColorSource::Dynamic(dynamic_component) => {
-                if let Some(Component::Color(color)) = dynamic_component.resolve(context) {
-                    color
-                } else {
-                    return false;
-                }
-            }
-        };
-        color.alpha() > 0
-    }
 }
 
 #[derive(Debug, PartialEq, Clone)]