@readium/navigator-html-injectables 2.4.2 → 2.4.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@readium/navigator-html-injectables",
-  "version": "2.4.2",
+  "version": "2.4.4",
   "type": "module",
   "description": "An embeddable solution for connecting frames of HTML publications with a Readium Navigator",
   "author": "readium",

package/src/helpers/dom.ts

@@ -79,6 +79,10 @@ export function isInteractiveElement(element: Element | null): boolean {
     // Check for interactive roles
     if (element.role && interactiveRoles.includes(element.role)) return true;
 
+    // An iframe element is a separate browsing context — its focused content cannot be inspected
+    // cross-origin, so the iframe container itself must not be treated as interactive.
+    if (element.tagName.toLowerCase() === "iframe") return false;
+
     if ((element as HTMLElement).tabIndex >= 0) return true;
     
     // Use existing interactive tags logic

package/src/keyboard/KeyCombinationManager.ts

@@ -146,6 +146,7 @@ export class KeyCombinationManager {
         const handlers = this.createKeyboardHandlers(targetFrameSrc, shortcuts, dispatcher, wnd);
 
         return (event: KeyboardEvent) => {
+            if (!event.isTrusted) return;
             for (const handlerConfig of handlers) {
                 if (this.match(event, [handlerConfig])) {
                     const suppress = handlerConfig.suppressOnInteractiveElement;

package/src/modules/Peripherals.ts

@@ -537,6 +537,15 @@ export class Peripherals extends Module {
             ack(true);
         });
 
+        // Disable keyboard handler when frame is unfocused so that stale focus
+        // inside a hidden/off-screen iframe doesn't silently swallow key events
+        // through a halted comms channel. The handler is re-established on the
+        // next show() cycle via the keyboard_peripherals message below.
+        this.comms?.register("unfocus", Peripherals.moduleName, (_, ack) => {
+            this.disableKeyboardPeripherals();
+            ack(true);
+        });
+
         // Separate handler for keyboard peripherals
         this.comms?.register("keyboard_peripherals", Peripherals.moduleName, (data: unknown, ack) => {
             const keyboardPeripherals = data as KeyboardPeripheral[];