refactor(ui): replace fromDocumentEvent with fromGlobalEvent (#1916)

* refactor(ui): replace fromDocumentEvent with fromGlobalEvent

* chore: fix lockfile

packages/data-validation/package.json

@@ -63,12 +63,12 @@
     },
     "peerDependencies": {
         "@univerjs/core": "workspace:*",
-        "@univerjs/protocol": "^0.1.14",
         "@univerjs/sheets": "workspace:*",
         "@wendellhu/redi": "0.13.0",
         "rxjs": ">=7.0.0"
     },
     "dependencies": {
+        "@univerjs/protocol": "^0.1.17",
         "dayjs": "^1.11.10"
     },
     "devDependencies": {

packages/facade/package.json

@@ -68,7 +68,6 @@
         "@univerjs/engine-render": "workspace:*",
         "@univerjs/network": "workspace:*",
         "@univerjs/sheets": "workspace:*",
-        "@univerjs/sheets-conditional-formatting": "workspace:*",
         "@univerjs/sheets-formula": "workspace:*",
         "@univerjs/sheets-numfmt": "workspace:*",
         "@univerjs/sheets-ui": "workspace:*",
@@ -82,6 +81,7 @@
         "@univerjs/network": "workspace:*",
         "@univerjs/shared": "workspace:*",
         "@univerjs/sheets": "workspace:*",
+        "@univerjs/sheets-conditional-formatting": "workspace:*",
         "@univerjs/sheets-formula": "workspace:*",
         "@univerjs/sheets-numfmt": "workspace:*",
         "@univerjs/sheets-ui": "workspace:*",

packages/sheets-conditional-formatting/package.json

@@ -67,19 +67,19 @@
         "@univerjs/core": "workspace:*",
         "@univerjs/engine-formula": "workspace:*",
         "@univerjs/engine-render": "workspace:*",
-        "@univerjs/protocol": "^0.1.14",
         "@univerjs/sheets": "workspace:*",
         "@wendellhu/redi": "0.13.0",
         "rxjs": ">=7.0.0"
     },
     "dependencies": {
+        "@univerjs/protocol": "^0.1.17",
         "dayjs": "^1.11.10"
     },
     "devDependencies": {
         "@univerjs/core": "workspace:*",
         "@univerjs/engine-formula": "workspace:*",
         "@univerjs/engine-render": "workspace:*",
-        "@univerjs/icons-svg": "^0.1.43",
+        "@univerjs/icons-svg": "^0.1.44",
         "@univerjs/shared": "workspace:*",
         "@univerjs/sheets": "workspace:*",
         "@wendellhu/redi": "^0.13.0",

packages/ui/src/common/lifecycle.ts

@@ -17,13 +17,13 @@
 import { toDisposable } from '@univerjs/core';
 import type { IDisposable } from '@wendellhu/redi';
 
-export function fromDocumentEvent<K extends keyof DocumentEventMap>(
+export function fromGlobalEvent<K extends keyof WindowEventMap>(
     type: K,
-    listener: (this: Document, ev: DocumentEventMap[K]) => any,
+    listener: (this: Window, ev: WindowEventMap[K]) => any,
     options?: boolean | AddEventListenerOptions
 ): IDisposable {
-    document.addEventListener(type, listener, options);
-    return toDisposable(() => document.removeEventListener(type, listener, options));
+    window.addEventListener(type, listener, options);
+    return toDisposable(() => window.removeEventListener(type, listener, options));
 }
 
 export function fromEvent<K extends keyof HTMLElementEventMap>(

packages/ui/src/services/shortcut/shortcut.service.ts

@@ -20,7 +20,7 @@ import { createIdentifier, Optional } from '@wendellhu/redi';
 import type { Observable } from 'rxjs';
 import { Subject } from 'rxjs';
 
-import { fromDocumentEvent } from '../../common/lifecycle';
+import { fromGlobalEvent } from '../../common/lifecycle';
 import { ILayoutService } from '../layout/layout.service';
 import { IPlatformService } from '../platform/platform.service';
 import { KeyCodeToChar, MetaKeys } from './keycode';
@@ -89,8 +89,10 @@ export class DesktopShortcutService extends Disposable implements IShortcutServi
 
         // Register native keydown event handler to trigger shortcuts.
         this.disposeWithMe(
-            fromDocumentEvent('keydown', (e: KeyboardEvent) => {
+            fromGlobalEvent('keydown', (e: KeyboardEvent) => {
                 this._resolveKeyboardEvent(e);
+            }, {
+                capture: true,
             })
         );
     }