fix(selectable): prevent clash with other's custom operations

- Make the `global_selection` operation check for the new
  `IndependentSelection` when trying to downcast_mut.
- `IndependentSelection` uses wraps a `RefCell` so it can change its
  inner value. Theres some helper methods to create this
  `IndependentSelection` from an existing value and to then get the
  changed value once the operation is done. This way the `Text` and
  `Rich` simply need to pass this `IndependentSelection` to the
  operation `custom` method which will make sure it wont clash with any
  other custom operations that could be using `&mut bool` as well.

2 files changed, 9 insertions, 7 deletions

diff --git a/src/text.rs b/src/text.rs
index 76560ce..4b789c9 100644
--- a/src/text.rs
+++ b/src/text.rs
@@ -752,8 +752,11 @@ where
             }
         }
 
-        // let mut isel = crate::operation::IndependentSelection::new(&mut self.independent_selection);
-        operation.custom(None, layout.bounds(), &mut self.independent_selection);
+        let mut isel = crate::operation::IndependentSelection::new(
+            self.independent_selection,
+        );
+        operation.custom(None, layout.bounds(), &mut isel);
+        self.independent_selection = isel.get();
     }
 
     fn mouse_interaction(
diff --git a/src/text/rich.rs b/src/text/rich.rs
index b49adbd..eb8b6b2 100644
--- a/src/text/rich.rs
+++ b/src/text/rich.rs
@@ -954,12 +954,11 @@ where
             }
         }
 
-        // let mut isel = crate::operation::IndependentSelection::new(&mut self.independent_selection);
-        operation.custom(
-            None,
-            layout.bounds(),
-            &mut self.independent_selection,
+        let mut isel = crate::operation::IndependentSelection::new(
+            self.independent_selection,
         );
+        operation.custom(None, layout.bounds(), &mut isel);
+        self.independent_selection = isel.get();
     }
 
     fn mouse_interaction(