fix(search-pad): clear selection when search opens

This ensures that all visual clutter is gone once the search pad is opened. Of course, on <Escape> the search restores.
bpmn-io · Nov 5, 2024 · 5d8eb9a · 5d8eb9a
1 parent 010d127
commit 5d8eb9a
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/lib/features/search-pad/SearchPad.js b/lib/features/search-pad/SearchPad.js
@@ -336,6 +336,8 @@ SearchPad.prototype.open = function() {
   this._cachedSelection = this._selection.get();
   this._cachedViewbox = this._canvas.viewbox();
 
+  this._selection.select(null);
+
   this._bindEvents();
 
   this._open = true;

diff --git a/test/spec/features/search-pad/SearchPadSpec.js b/test/spec/features/search-pad/SearchPadSpec.js
@@ -192,6 +192,19 @@ describe('features/search-pad', function() {
     }));
 
 
+    it('should clear selection', inject(function(searchPad, selection) {
+
+      // given
+      selection.select(elements.one.a);
+
+      // when
+      searchPad.open();
+
+      // then
+      expect(selection.get()).to.be.empty;
+    }));
+
+
     it('should error when provider not registered', inject(function(searchPad) {
 
       // given