@applicaster/zapp-react-dom-app 13.0.0-alpha.1632352572 → 13.0.0-alpha.1692038547

package/App/InteractionManager/index.tsx

@@ -140,13 +140,16 @@ class InteractionManagerClass extends React.Component<Props, State> {
     this.onConfirmDialogOpen = this.onConfirmDialogOpen.bind(this);
     this.onConfirmDialogClose = this.onConfirmDialogClose.bind(this);
 
+    this.handlePhysicalKeyboardDismiss =
+      this.handlePhysicalKeyboardDismiss.bind(this);
+
     this.confirmDialog.setConfirmAction(this.performExit);
     this.confirmDialog.setCancelAction(this.onConfirmDialogClose);
     this.longPressDetector = new KeyboardLongPressManager();
   }
 
   componentDidMount() {
-    window.addEventListener("keydown", this.onKeyDownListener);
+    window.addEventListener("keydown", this.onKeyDownListener, true);
     window.addEventListener("keyup", this.onKeyUp);
     window.addEventListener("mousemove", this.onMouseMoveListener);
 
@@ -181,6 +184,11 @@ class InteractionManagerClass extends React.Component<Props, State> {
       "onLongPressRelease",
       this.handleLongPressRelease.bind(this)
     );
+
+    document.addEventListener(
+      "keyboardDismissedByPhysicalKeypress",
+      this.handlePhysicalKeyboardDismiss
+    );
   }
 
   componentWillUnmount() {
@@ -209,6 +217,11 @@ class InteractionManagerClass extends React.Component<Props, State> {
       "onLongPressRelease",
       this.handleLongPressRelease
     );
+
+    document.removeEventListener(
+      "keyboardDismissedByPhysicalKeypress",
+      this.handlePhysicalKeyboardDismiss
+    );
   }
 
   performExit() {
@@ -341,8 +354,10 @@ class InteractionManagerClass extends React.Component<Props, State> {
       this.longPressDetector.endKeyPressMonitoring(event.code);
     }
 
-    if (keyCode(event).matches(KEYS.Enter) ||
-      (__DEV__ && keyCode(event).matches(KEYS.X))) {
+    if (
+      keyCode(event).matches(KEYS.Enter) ||
+      (__DEV__ && keyCode(event).matches(KEYS.X))
+    ) {
       focusManager.pressOut();
     }
 
@@ -384,8 +399,8 @@ class InteractionManagerClass extends React.Component<Props, State> {
       this.longPressDetector.monitorKeyPress(event.code, event);
     }
 
-    // None of the logic below should be executed for repeated keys
-    if (event.repeat) return;
+    // Only handle repeat events for arrow keys
+    if (event.repeat && !keyCode(event).matchesAny(...ARROW_KEYS)) return;
 
     /**
      When the player is displayed, we listen to Native Play / Pause / Stop
@@ -446,20 +461,13 @@ class InteractionManagerClass extends React.Component<Props, State> {
     }
 
     /**
-     When pressing backspace or the Native "Back" button:
+     When pressing escape or the Native "Back" button:
      if on the transport controls: we show the player
      if on the player : we show the UI
      if on the UI : we exit the app
+     note: backspace affects the keyboard input, so it not handled here
      */
-
-    if (
-      keyCode(event).matchesAny(
-        KEYS.Back,
-        KEYS.Escape,
-        KEYS.EscapeDevice,
-        KEYS.Backspace
-      )
-    ) {
+    if (keyCode(event).matchesAny(KEYS.Back, KEYS.Escape, KEYS.EscapeDevice)) {
       switch (displayState) {
         case DISPLAY_STATES.DEFAULT:
         case DISPLAY_STATES.PLAYER:
@@ -477,7 +485,9 @@ class InteractionManagerClass extends React.Component<Props, State> {
             const isHookInHomescreen = entryData?.payload?.home;
 
             const isHookFlowBlocker =
-              entryData?.hookPlugin?.module?.isFlowBlocker();
+              typeof entryData?.hookPlugin?.module?.isFlowBlocker === "function"
+                ? entryData?.hookPlugin?.module?.isFlowBlocker()
+                : entryData?.hookPlugin?.module?.isFlowBlocker;
 
             if (isHookInHomescreen && isHookFlowBlocker) {
               this.platformBack();
@@ -499,10 +509,14 @@ class InteractionManagerClass extends React.Component<Props, State> {
       }
     }
 
-    if (keyCode(event).matches(KEYS.Enter) || keyCode(event).matches(KEYS.X)) {
+    if (
+      keyCode(event).matches(KEYS.Enter) ||
+      (__DEV__ && keyCode(event).matches(KEYS.X))
+    ) {
       if (displayState === DISPLAY_STATES.PLAYER) {
         setDisplayState(DISPLAY_STATES.HUD);
       } else {
+        focusManager.pressIn();
         focusManager.press();
       }
 
@@ -535,12 +549,6 @@ class InteractionManagerClass extends React.Component<Props, State> {
         focusManager.moveFocus(direction);
       }
 
-      if (!isDialogVisible) {
-        // run focus recovery on user interaction attempt
-        // probably we failed to set initial focus for some reason
-        focusManager.recoverFocus();
-      }
-
       if (
         displayState === DISPLAY_STATES.PLAYER ||
         displayState === DISPLAY_STATES.HUD
@@ -616,6 +624,7 @@ class InteractionManagerClass extends React.Component<Props, State> {
 
     if (
       (target.tagName === "INPUT" && target.type === "text") ||
+      (target.tagName === "INPUT" && target.type === "password") ||
       target.tagName === "TEXTAREA"
     ) {
       this.setState({
@@ -644,7 +653,7 @@ class InteractionManagerClass extends React.Component<Props, State> {
     }
   }
 
-  handleKeyboardDismiss = () => {
+  handleKeyboardDismiss() {
     this.isDismissing = true;
 
     this.setState(
@@ -655,9 +664,10 @@ class InteractionManagerClass extends React.Component<Props, State> {
       },
       () => {
         this.isDismissing = false;
+        focusManager.recoverFocus();
       }
     );
-  };
+  }
 
   handleKeyboardInput = (value: string) => {
     const { activeInputRef } = this.state;
@@ -717,10 +727,6 @@ class InteractionManagerClass extends React.Component<Props, State> {
   // TODO: Delete it on next PR if not needed
   private handleLongPress = (event: CustomEvent) => {
     const { keyCode } = event.detail;
-    const { displayState } = this.props;
-    const playerState = displayState === DISPLAY_STATES.PLAYER;
-    const hudState = displayState === DISPLAY_STATES.HUD;
-    const playbackState = playerState || hudState;
     this.longPressActive = true;
     this.cancelHudTimeout();
 
@@ -757,6 +763,35 @@ class InteractionManagerClass extends React.Component<Props, State> {
     ].includes(event.code);
   }
 
+  handlePhysicalKeyboardDismiss(event: CustomEvent) {
+    const { value, position, inputElement } = event.detail;
+
+    this.isDismissing = true;
+
+    this.setState(
+      {
+        isKeyboardVisible: false,
+        keyboardInput: "",
+        activeInputRef: null,
+      },
+      () => {
+        if (inputElement) {
+          inputElement.value = value;
+          inputElement.focus();
+          inputElement.setSelectionRange(position, position);
+
+          const keyboardEvent = new CustomEvent("keyboardUpdate", {
+            detail: { value },
+          });
+
+          document.dispatchEvent(keyboardEvent);
+        }
+
+        this.isDismissing = false;
+      }
+    );
+  }
+
   render() {
     return (
       (shouldUseOnScreenKeyboard() && this.state.isKeyboardVisible && (

package/App/Loader/__tests__/__snapshots__/loadDomPlugins.test.js.snap

@@ -1,15 +1,15 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`loadDomPlugins when there are plugins requiring to write to the dom skips plugins without init method 1`] = `
-Array [
-  Array [
+[
+  [
     "could not initialize plugin Plugin with Dom node & no init method - it doesn't have an init method",
-    Object {
-      "plugin": Object {
-        "configuration": Object {
+    {
+      "plugin": {
+        "configuration": {
           "foo": "bar",
         },
-        "module": Object {
+        "module": {
           "requiresAppendingDomNode": true,
         },
         "name": "Plugin with Dom node & no init method",

package/App/Loader/loadLocale.js

@@ -4,8 +4,8 @@ import { QuickBrickCommunicationModule } from "../../Polyfills/QuickBrickCommuni
 import { loadLocaleImpl, getLanguageCode } from "./utils";
 
 type Locale = {
-  language: string,
-  country: string,
+  language: string;
+  country: string;
 };
 
 async function getBrowserLocale() {

package/App/Loader/utils/platform.js

@@ -11,11 +11,28 @@ import {
   PLATFORMS,
 } from "./const";
 
+/**
+ * @typedef {"tv" | "web" | "other"} DeviceType
+ * @typedef {"samsung_tv" | "lg_tv" | unknown} CustomPlatformTypes
+ */
+
+/**
+ * Retrieves the user agent string from the browser or global navigator.
+ * @returns {string} The user agent string.
+ */
 export const getUserAgent = () =>
   window?.navigator?.userAgent || global?.navigator?.userAgent;
 
+/**
+ * Checks if the device is HD (1280 width).
+ * @returns {boolean} True if the device is HD, false otherwise.
+ */
 export const isHD = () => window.innerWidth === 1280;
 
+/**
+ * Checks if the platform is LG.
+ * @returns {boolean} True if the platform is LG, false otherwise.
+ */
 export const isLgPlatform = () => {
   const userAgent = getUserAgent();
 
@@ -23,10 +40,13 @@ export const isLgPlatform = () => {
     return false;
   }
 
-  // Note: LG uses the number 0 in their userAgent, possibly because of Palm WebOS
   return hasWebOS || userAgent.includes(PLATFORMS.webos);
 };
 
+/**
+ * Checks if the platform is Samsung.
+ * @returns {boolean} True if the platform is Samsung, false otherwise.
+ */
 export const isSamsungPlatform = () => {
   const userAgent = getUserAgent();
 
@@ -42,42 +62,39 @@ export const isSamsungPlatform = () => {
 };
 
 /**
- * Checks if we are on the Vizio platform by using the userAgent string
- *
- * Use this method when you need to identify the platform on runtime
- * before the VIZIO Companion Library has loaded
+ * Checks if the platform is Vizio by using the user agent string.
+ * @returns {boolean} True if the platform is Vizio, false otherwise.
  */
 export const isVizioPlatform = () => {
   const userAgent = getUserAgent();
 
-  const isVizioAgent =
+  return (
     userAgent.includes(PLATFORMS.vizio) ||
     userAgent.includes(PLATFORMS.smartcast) ||
-    userAgent.includes(PLATFORMS.conjure);
-
-  return isVizioAgent;
+    userAgent.includes(PLATFORMS.conjure)
+  );
 };
 
 /**
- * Checks if we are on the Vizio platform by checking for the global VIZIO object
- * and the VIZIO Companion Library. This method does not use the userAgent string,
- * because it mainly checks whether we have access to apis, which may not be available yet
+ * Checks if the Vizio APIs are available.
+ * @returns {boolean} True if the Vizio APIs are available, false otherwise.
  */
 export const hasVizioAPIs = () => {
-  const hasAPIs =
+  return (
     typeof window.VIZIO !== "undefined" &&
-    window?.applicaster?.vizioLibraryDidLoad;
-
-  return hasAPIs;
+    window?.applicaster?.vizioLibraryDidLoad
+  );
 };
 
 /**
- * platformSelect can't be used, because Platform.OS is overriden
- * by overloadReactNativePlatform and don't support 'web' value
- * @returns {"tv" | "web" | "other"} value
+ * Determines the device type based on the platform.
+ * @returns {DeviceType} The device type ("tv", "web", or "other").
  */
 export const getDeviceType = () => {
-  switch (Platform.OS) {
+  /** @type {CustomPlatformTypes} */
+  const platformOS = Platform.OS;
+
+  switch (platformOS) {
     case "samsung_tv":
       return isSamsungPlatform() ? "tv" : "web";
     case "lg_tv":
@@ -88,14 +105,15 @@ export const getDeviceType = () => {
 };
 
 /**
- * This function is used to rename keys
- * @param {string} key - key you would like to rename
+ * Renames keys based on a predefined map.
+ * @param {string} key - The key to rename.
+ * @returns {string} The renamed key.
  */
 export const renameKeys = (key) => desiredKeysMap?.[key] || key;
 
 /**
- * This function is used to get the connection info from WebOS
- * TODO: move to connection hook
+ * Retrieves the connection info from WebOS.
+ * @returns {Promise<{networkType: string} | null>} A promise that resolves with the connection info or null.
  */
 export const getWebOSConnectionInfo = () => {
   return new Promise((resolve, reject) => {
@@ -108,7 +126,6 @@ export const getWebOSConnectionInfo = () => {
     webOS.service.request("luna://com.palm.connectionmanager", {
       method: "getStatus",
       parameters: { subscribe: false },
-
       onSuccess: (args) => {
         const types = Object.keys(args)
           .filter((key) => args[key].state === "connected")
@@ -116,7 +133,6 @@ export const getWebOSConnectionInfo = () => {
 
         resolve({ networkType: types });
       },
-
       onFailure: (error) => {
         console.error(
           "getWebOSConnectionInfo: There was an error getting connection info",
@@ -129,6 +145,10 @@ export const getWebOSConnectionInfo = () => {
   });
 };
 
+/**
+ * Retrieves the connection info from Tizen.
+ * @returns {Promise<{networkType: string} | null>} A promise that resolves with the connection info or null.
+ */
 export const getTizenConnectionInfo = () => {
   return new Promise((resolve, reject) => {
     if (!isSamsungPlatform) {

package/Polyfills/Storage/__tests__/__snapshots__/storage.test.js.snap

@@ -1,7 +1,7 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`storageObject when the requested storage exists getAllItems returns all keys 1`] = `
-Object {
+{
   "baz_::_foo": "baz",
   "baz_::_otherKey": "otherValue",
   "foo": "bar",
@@ -10,7 +10,7 @@ Object {
 `;
 
 exports[`storageObject when the requested storage exists getAllItems returns all keys 2`] = `
-Object {
+{
   "baz_::_foo": "baz",
   "baz_::_otherKey": "otherValue",
   "foo": "bar",
@@ -19,14 +19,14 @@ Object {
 `;
 
 exports[`storageObject when the requested storage exists getAllItems returns all keys in the namespace if provided 1`] = `
-Object {
+{
   "foo": "baz",
   "otherKey": "otherValue",
 }
 `;
 
 exports[`storageObject when the requested storage exists getAllItems returns all keys in the namespace if provided 2`] = `
-Object {
+{
   "foo": "baz",
   "otherKey": "otherValue",
 }

package/index.js

@@ -1,13 +1,11 @@
 /// <reference types="@applicaster/applicaster-types" />
 import * as R from "ramda";
-import { useHistory, useLocation } from "react-router-dom";
 
 import { createQuickBrickApp } from "@applicaster/quick-brick-core";
 import * as QuickBrickManager from "@applicaster/zapp-react-native-bridge/QuickBrick";
 import * as Contexts from "@applicaster/zapp-react-native-ui-components/Contexts";
 import { OfflineHandler } from "@applicaster/zapp-react-native-ui-components/Components/OfflineHandler";
 import { getXray } from "@applicaster/zapp-react-native-utils/logger";
-import { ZappDomRouter } from "./App/Router";
 import { Layout } from "./App/Layout";
 import { SplashLoader } from "./App/Loader/SplashLoader";
 import { appLoader } from "./App/Loader";
@@ -24,11 +22,6 @@ const contextProviders = R.map(R.prop("withProvider"), R.values(Contexts));
 if (withXray) contextProviders.push(withXray);
 
 export const ZappApp = createQuickBrickApp({
-  navigations: {
-    Router: ZappDomRouter,
-    useLocation,
-    useHistory,
-  },
   QuickBrickManager,
   Layout,
   OfflineHandler,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@applicaster/zapp-react-dom-app",
-  "version": "13.0.0-alpha.1632352572",
+  "version": "13.0.0-alpha.1692038547",
   "description": "Zapp App Component for Applicaster's Quick Brick React Native App",
   "main": "index.js",
   "scripts": {
@@ -22,13 +22,12 @@
   },
   "homepage": "https://github.com/applicaster/zapp-react-dom-app#readme",
   "dependencies": {
-    "@applicaster/zapp-react-dom-ui-components": "13.0.0-alpha.1632352572",
-    "@applicaster/zapp-react-native-bridge": "13.0.0-alpha.1632352572",
-    "@applicaster/zapp-react-native-redux": "13.0.0-alpha.1632352572",
-    "@applicaster/zapp-react-native-ui-components": "13.0.0-alpha.1632352572",
-    "@applicaster/zapp-react-native-utils": "13.0.0-alpha.1632352572",
+    "@applicaster/zapp-react-dom-ui-components": "13.0.0-alpha.1692038547",
+    "@applicaster/zapp-react-native-bridge": "13.0.0-alpha.1692038547",
+    "@applicaster/zapp-react-native-redux": "13.0.0-alpha.1692038547",
+    "@applicaster/zapp-react-native-ui-components": "13.0.0-alpha.1692038547",
+    "@applicaster/zapp-react-native-utils": "13.0.0-alpha.1692038547",
     "abortcontroller-polyfill": "^1.7.5",
-    "history": "^4.7.2",
     "react-router-dom": "^5.1.2",
     "typeface-montserrat": "^0.0.54",
     "video.js": "7.14.3",
@@ -36,9 +35,19 @@
     "videojs-contrib-quality-levels": "3.0.0"
   },
   "peerDependencies": {
+    "@applicaster/zapp-pipes-v2-client": "*",
+    "@react-native-community/netinfo": "*",
+    "@types/node": "*",
     "core-js": "3.29.1",
+    "immer": "*",
     "react": "17.0.2",
     "react-dom": "17.0.2",
-    "react-native-web": "0.18.11"
+    "react-native": "*",
+    "react-native-safe-area-context": "*",
+    "react-native-svg": "*",
+    "react-native-web": "0.18.11",
+    "uglify-js": "*",
+    "validate-color": "*",
+    "zustand": "*"
   }
 }

package/App/Router/index.js

@@ -1,17 +0,0 @@
-import React from "react";
-import PropTypes from "prop-types";
-
-import { Router } from "react-router";
-import { createMemoryHistory } from "history";
-
-const history = createMemoryHistory();
-
-export const ZappDomRouter = ({ children, ...otherProps }) => (
-  <Router history={history} {...otherProps}>
-    {children}
-  </Router>
-);
-
-ZappDomRouter.propTypes = {
-  children: PropTypes.node.isRequired,
-};

package/App/Splash/Spinner.js

@@ -1,30 +0,0 @@
-import React from "react";
-
-import "./spinner-styles.css";
-
-export const Spinner = (color) => {
-  const spinnerColor = {
-    background: color.color,
-  };
-
-  return (
-    <div>
-      <div id="progress">
-        <div className="spinner">
-          <div style={spinnerColor} className="bar1" />
-          <div style={spinnerColor} className="bar2" />
-          <div style={spinnerColor} className="bar3" />
-          <div style={spinnerColor} className="bar4" />
-          <div style={spinnerColor} className="bar5" />
-          <div style={spinnerColor} className="bar6" />
-          <div style={spinnerColor} className="bar7" />
-          <div style={spinnerColor} className="bar8" />
-          <div style={spinnerColor} className="bar9" />
-          <div style={spinnerColor} className="bar10" />
-          <div style={spinnerColor} className="bar11" />
-          <div style={spinnerColor} className="bar12" />
-        </div>
-      </div>
-    </div>
-  );
-};

package/App/Splash/spinner-styles.css

@@ -1,81 +0,0 @@
-div.spinner {
-  position: absolute;
-  width: 100px;
-  height: 100px;
-  display: inline-block;
-  top: 50%;
-  left: 50%;
-  margin: -60px 0 0 -70px;
-}
-
-div.spinner div {
-  width: 6%;
-  height: 16%;
-  position: absolute;
-  left: 49%;
-  top: 43%;
-  opacity: 0;
-  -webkit-border-radius: 50px;
-  -webkit-box-shadow: 0 0 3px rgba(0, 0, 0, 0.2);
-  -webkit-animation: fade 1s linear infinite;
-}
-
-@-webkit-keyframes fade {
-  from {
-    opacity: 1;
-  }
-  to {
-    opacity: 0.25;
-  }
-}
-
-div.spinner div.bar1 {
-  -webkit-transform: rotate(0deg) translate(0, -130%);
-  -webkit-animation-delay: 0s;
-}
-
-div.spinner div.bar2 {
-  -webkit-transform: rotate(30deg) translate(0, -130%);
-  -webkit-animation-delay: -0.9167s;
-}
-
-div.spinner div.bar3 {
-  -webkit-transform: rotate(60deg) translate(0, -130%);
-  -webkit-animation-delay: -0.833s;
-}
-div.spinner div.bar4 {
-  -webkit-transform: rotate(90deg) translate(0, -130%);
-  -webkit-animation-delay: -0.7497s;
-}
-div.spinner div.bar5 {
-  -webkit-transform: rotate(120deg) translate(0, -130%);
-  -webkit-animation-delay: -0.667s;
-}
-div.spinner div.bar6 {
-  -webkit-transform: rotate(150deg) translate(0, -130%);
-  -webkit-animation-delay: -0.5837s;
-}
-div.spinner div.bar7 {
-  -webkit-transform: rotate(180deg) translate(0, -130%);
-  -webkit-animation-delay: -0.5s;
-}
-div.spinner div.bar8 {
-  -webkit-transform: rotate(210deg) translate(0, -130%);
-  -webkit-animation-delay: -0.4167s;
-}
-div.spinner div.bar9 {
-  -webkit-transform: rotate(240deg) translate(0, -130%);
-  -webkit-animation-delay: -0.333s;
-}
-div.spinner div.bar10 {
-  -webkit-transform: rotate(270deg) translate(0, -130%);
-  -webkit-animation-delay: -0.2497s;
-}
-div.spinner div.bar11 {
-  -webkit-transform: rotate(300deg) translate(0, -130%);
-  -webkit-animation-delay: -0.167s;
-}
-div.spinner div.bar12 {
-  -webkit-transform: rotate(330deg) translate(0, -130%);
-  -webkit-animation-delay: -0.0833s;
-}