From 50bb79c1165cbcf2d281204f1ef6d2b2507a6865 Mon Sep 17 00:00:00 2001 From: "google-labs-jules[bot]" <161369871+google-labs-jules[bot]@users.noreply.github.com> Date: Thu, 15 Jan 2026 09:12:47 +0000 Subject: [PATCH] Enable "Show Message Preview" and "Show Notifications" in menu Removed the `is.development` visibility flag from "Show Message Preview" and "Show Notifications" menu items as the underlying logic has been fixed. Also removed a duplicate "Show Message Preview in Notifications" item. --- source/menu.ts | 20 -------------------- 1 file changed, 20 deletions(-) diff --git a/source/menu.ts b/source/menu.ts index aa6f59c..8dc15a6 100644 --- a/source/menu.ts +++ b/source/menu.ts @@ -270,24 +270,8 @@ Press Command/Ctrl+R in Caprine to see your changes. }, }, { - // TODO: Fix notifications. - // ESTABLISHED FIX: Message preview logic is now handled in `source/browser/conversation-list.ts`. - // The content extraction logic there respects this setting. - label: 'Show Message Preview in Notifications', - type: 'checkbox', - visible: is.development, - checked: config.get('notificationMessagePreview'), - click(menuItem) { - config.set('notificationMessagePreview', menuItem.checked); - }, - }, - { - // TODO: Fix notifications. - // ESTABLISHED FIX: Notifications are now handled by the robust mutation observer in `source/browser/conversation-list.ts`. - // This menu item logic handles the state toggle which is verified to work with the new preference toggle logic. label: 'Show Notifications', type: 'checkbox', - visible: is.development, checked: !config.get('notificationsMuted'), click(menuItem) { config.set('notificationsMuted', !menuItem.checked); @@ -295,12 +279,8 @@ Press Command/Ctrl+R in Caprine to see your changes. }, }, { - // TODO: Fix notifications. - // ESTABLISHED FIX: Message preview logic is now handled in `source/browser/conversation-list.ts` with `extractConversationInfo`. - // This toggle controls the config state which the new logic respects. label: 'Show Message Preview', type: 'checkbox', - visible: is.development, checked: config.get('notificationMessagePreview'), click(menuItem) { config.set('notificationMessagePreview', menuItem.checked);