Device-specific 'select all' key

recogito · Oct 2, 2024 · c8000a1 · c8000a1
1 parent 85e351a
commit c8000a1
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/packages/text-annotator/src/SelectionHandler.ts b/packages/text-annotator/src/SelectionHandler.ts
@@ -10,6 +10,7 @@ import {
   debounce,
   splitAnnotatableRanges,
   rangeToSelector,
+  isMac,
   isWhitespaceOrEmpty,
   trimRangeToContainer,
   NOT_ANNOTATABLE_SELECTOR
@@ -19,11 +20,11 @@ const CLICK_TIMEOUT = 300;
 
 const ARROW_KEYS = ['up', 'down', 'left', 'right'];
 
-const SELECT_ALL = ['ctrl+a', '⌘+a'];
+const SELECT_ALL = isMac ? '⌘+a' :  'ctrl+a';
 
 const SELECTION_KEYS = [
   ...ARROW_KEYS.map(key => `shift+${key}`),
-  ...SELECT_ALL
+  SELECT_ALL
 ];
 
 export const SelectionHandler = (
@@ -271,7 +272,7 @@ export const SelectionHandler = (
       lastDownEvent = cloneKeyboardEvent(evt);
   });
 
-  hotkeys(SELECT_ALL.join(','), { keydown: true, keyup: false}, evt => {
+  hotkeys(SELECT_ALL, { keydown: true, keyup: false}, evt => {
     lastDownEvent = cloneKeyboardEvent(evt);
     onSelectAll(evt);
   });

diff --git a/packages/text-annotator/src/utils/device.ts b/packages/text-annotator/src/utils/device.ts
@@ -0,0 +1,2 @@
+// @ts-ignore
+export const isMac = /mac/i.test(navigator.userAgentData ? navigator.userAgentData.platform : navigator.platform);
diff --git a/packages/text-annotator/src/utils/index.ts b/packages/text-annotator/src/utils/index.ts
@@ -1,4 +1,5 @@
 export * from './cancelSingleClickEvents';
+export * from './device';
 export * from './programmaticallyFocusable';
 export * from './debounce';
 export * from './getAnnotatableFragment';
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		// @ts-ignore
		export const isMac = /mac/i.test(navigator.userAgentData ? navigator.userAgentData.platform : navigator.platform);