separate context menu from customizer menu

Author: shakyShane

special-pages/pages/new-tab/app/components/App.js

@@ -4,14 +4,15 @@ import styles from './App.module.css';
 import { useCustomizerDrawerSettings, usePlatformName } from '../settings.provider.js';
 import { WidgetList } from '../widget-list/WidgetList.js';
 import { useGlobalDropzone } from '../dropzone.js';
-import { CustomizerButton, CustomizerMenuPositionedFixed, useContextMenu } from '../customizer/components/CustomizerMenu.js';
+import { CustomizerButton, CustomizerMenuPositionedFixed } from '../customizer/components/CustomizerMenu.js';
 import { useDrawer, useDrawerControls } from './Drawer.js';
 import { CustomizerDrawer } from '../customizer/components/CustomizerDrawer.js';
 import { BackgroundConsumer } from './BackgroundProvider.js';
 import { useComputed } from '@preact/signals';
 import { CustomizerThemesContext } from '../customizer/CustomizerProvider.js';
 import { useContext } from 'preact/hooks';
 import { InlineErrorBoundary } from '../InlineErrorBoundary.js';
+import { useContextMenu } from '../context-menu.js';
 
 /**
  * Renders the App component.

special-pages/pages/new-tab/app/context-menu.js

@@ -0,0 +1,73 @@
+import { useMessaging } from './types.js';
+import { useEffect } from 'preact/hooks';
+
+// eslint-disable-next-line @typescript-eslint/no-unused-vars
+class ContextMenuItem {
+    /** @type {string} */
+    title;
+    /** @type {string} */
+    id;
+    /** @type {number} */
+    index;
+}
+
+const OPEN_EVENT = 'ntp-contextMenu-open';
+
+/**
+ * @returns {ContextMenuItem[]}
+ */
+export function collect() {
+    /** @type {ContextMenuItem[]} */
+    const next = [];
+    const detail = {
+        register: (/** @type {ContextMenuItem} */ incoming) => {
+            next.push(incoming);
+        },
+    };
+    const event = new CustomEvent(OPEN_EVENT, { detail });
+    window.dispatchEvent(event);
+    next.sort((a, b) => a.index - b.index);
+    return next;
+}
+
+/**
+ * Forward the contextmenu event
+ */
+export function useContextMenu() {
+    const messaging = useMessaging();
+    useEffect(() => {
+        function handler(e) {
+            e.preventDefault();
+            e.stopImmediatePropagation();
+            const items = collect();
+            /** @type {Omit<ContextMenuItem, "index">[]} */
+            const simplified = items
+                .filter((x) => !x.id.startsWith('_'))
+                .map((item) => {
+                    return {
+                        id: item.id,
+                        title: item.title,
+                    };
+                });
+            messaging.contextMenu({ visibilityMenuItems: simplified });
+        }
+        document.body.addEventListener('contextmenu', handler);
+        return () => {
+            document.body.removeEventListener('contextmenu', handler);
+        };
+    }, [messaging]);
+}
+
+/**
+ * Call this to opt-in to the visibility menu
+ * @param {ContextMenuItem} row
+ */
+export function useContextMenuItem({ title, id, index }) {
+    useEffect(() => {
+        const handler = (/** @type {CustomEvent<any>} */ e) => {
+            e.detail.register({ title, id, index });
+        };
+        window.addEventListener(OPEN_EVENT, handler);
+        return () => window.removeEventListener(OPEN_EVENT, handler);
+    }, [title, id, index]);
+}

special-pages/pages/new-tab/app/customizer/components/CustomizerMenu.js

@@ -2,7 +2,7 @@ import { h } from 'preact';
 import { useEffect } from 'preact/hooks';
 import styles from './Customizer.module.css';
 import { CustomizeIcon } from '../../components/Icons.js';
-import { useMessaging, useTypedTranslation } from '../../types.js';
+import { useTypedTranslation } from '../../types.js';
 
 /**
  * @import { WidgetVisibility, VisibilityMenuItem } from '../../../types/new-tab.js'
@@ -21,7 +21,10 @@ import { useMessaging, useTypedTranslation } from '../../types.js';
 export const OPEN_EVENT = 'ntp-customizer-open';
 export const UPDATE_EVENT = 'ntp-customizer-update';
 
-export function getItems() {
+/**
+ * @returns {VisibilityRowData[]}
+ */
+export function getCustomizerItems() {
     /** @type {VisibilityRowData[]} */
     const next = [];
     const detail = {
@@ -35,34 +38,6 @@ export function getItems() {
     return next;
 }
 
-/**
- * Forward the contextmenu event
- */
-export function useContextMenu() {
-    const messaging = useMessaging();
-    useEffect(() => {
-        function handler(e) {
-            e.preventDefault();
-            e.stopImmediatePropagation();
-            const items = getItems();
-            /** @type {VisibilityMenuItem[]} */
-            const simplified = items
-                .filter((x) => !x.id.startsWith('_'))
-                .map((item) => {
-                    return {
-                        id: item.id,
-                        title: item.title,
-                    };
-                });
-            messaging.contextMenu({ visibilityMenuItems: simplified });
-        }
-        document.body.addEventListener('contextmenu', handler);
-        return () => {
-            document.body.removeEventListener('contextmenu', handler);
-        };
-    }, [messaging]);
-}
-
 /**
  * @param {object} props
  * @param {string} [props.menuId]

special-pages/pages/new-tab/app/customizer/components/VisibilityMenuSection.js

@@ -1,16 +1,16 @@
 import { useLayoutEffect, useState } from 'preact/hooks';
-import { getItems, UPDATE_EVENT } from './CustomizerMenu.js';
+import { getCustomizerItems, UPDATE_EVENT } from './CustomizerMenu.js';
 import { EmbeddedVisibilityMenu } from './VisibilityMenu.js';
 import { h } from 'preact';
 
 export function VisibilityMenuSection() {
     const [rowData, setRowData] = useState(() => {
-        const items = /** @type {import("./CustomizerMenu.js").VisibilityRowData[]} */ (getItems());
+        const items = /** @type {import("./CustomizerMenu.js").VisibilityRowData[]} */ (getCustomizerItems());
         return items;
     });
     useLayoutEffect(() => {
         function handler() {
-            setRowData(getItems());
+            setRowData(getCustomizerItems());
         }
 
         window.addEventListener(UPDATE_EVENT, handler);

special-pages/pages/new-tab/app/favorites/components/FavoritesCustomized.js

@@ -11,6 +11,7 @@ import { FavoritesMemo } from './Favorites.js';
 import { viewTransition } from '../../utils.js';
 import { CustomizerContext } from '../../customizer/CustomizerProvider.js';
 import { Shield } from '../../components/Icons.js';
+import { useContextMenuItem } from '../../context-menu.js';
 
 /**
  * @typedef {import('../../../types/new-tab.ts').Favorite} Favorite
@@ -68,6 +69,7 @@ export function FavoritesCustomized() {
     // register with the visibility menu
     const title = t('favorites_menu_title');
     useCustomizer({ title, id, icon: <Shield />, toggle, visibility: visibility.value, index });
+    useContextMenuItem({ title, id, index });
 
     if (visibility.value === 'hidden') {
         return null;

special-pages/pages/new-tab/app/omnibar/components/OmnibarCustomized.js

@@ -6,6 +6,7 @@ import { h } from 'preact';
 
 import { OmnibarConsumer } from './OmnibarConsumer.js';
 import { SearchIcon } from '../../components/Icons.js';
+import { useContextMenuItem } from '../../context-menu.js';
 
 /**
  * @import enStrings from "../strings.json"
@@ -29,6 +30,12 @@ export function OmnibarCustomized() {
     const { visibility, id, toggle, index } = useVisibility();
 
     useCustomizer({ title: sectionTitle, id, icon: <SearchIcon />, toggle, visibility: visibility.value, index });
+    useContextMenuItem({ title: sectionTitle, id, index });
+    useContextMenuItem({
+        title: t('omnibar_toggleDuckAi'),
+        id: `${id}-toggleAi`,
+        index: index + 0.1,
+    });
 
     if (visibility.value === 'hidden') {
         return null;

special-pages/pages/new-tab/app/protections/components/ProtectionsCustomized.js

@@ -6,6 +6,7 @@ import { h } from 'preact';
 
 import { ProtectionsConsumer } from './ProtectionsConsumer.js';
 import { DuckFoot } from '../../components/Icons.js';
+import { useContextMenuItem } from '../../context-menu.js';
 
 /**
  * @import enStrings from "../strings.json"
@@ -29,6 +30,7 @@ export function ProtectionsCustomized() {
     const { visibility, id, toggle, index } = useVisibility();
 
     useCustomizer({ title: sectionTitle, id, icon: <DuckFoot />, toggle, visibility: visibility.value, index });
+    useContextMenuItem({ title: sectionTitle, id, index });
 
     if (visibility.value === 'hidden') {
         return null;