From 26dfae00ce1413472b4ec203df8dea6c49546e6c Mon Sep 17 00:00:00 2001 From: Mircea Hasegan Date: Thu, 2 Oct 2025 17:21:57 +0200 Subject: [PATCH 1/3] fix(extension): add notification icon for bitcoin layout --- .../bitcoin-mode/components/MainMenu/MainHeader.module.scss | 2 +- .../src/views/bitcoin-mode/components/MainMenu/MainHeader.tsx | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/apps/browser-extension-wallet/src/views/bitcoin-mode/components/MainMenu/MainHeader.module.scss b/apps/browser-extension-wallet/src/views/bitcoin-mode/components/MainMenu/MainHeader.module.scss index b2b8c78ff6..689c0e2acf 100644 --- a/apps/browser-extension-wallet/src/views/bitcoin-mode/components/MainMenu/MainHeader.module.scss +++ b/apps/browser-extension-wallet/src/views/bitcoin-mode/components/MainMenu/MainHeader.module.scss @@ -27,7 +27,7 @@ .controls { display: flex; align-items: center; - gap: size_unit(1); + gap: size_unit(0.75); .avatar { font-size: 20px; diff --git a/apps/browser-extension-wallet/src/views/bitcoin-mode/components/MainMenu/MainHeader.tsx b/apps/browser-extension-wallet/src/views/bitcoin-mode/components/MainMenu/MainHeader.tsx index 5fad3331f3..a38ec54af8 100644 --- a/apps/browser-extension-wallet/src/views/bitcoin-mode/components/MainMenu/MainHeader.tsx +++ b/apps/browser-extension-wallet/src/views/bitcoin-mode/components/MainMenu/MainHeader.tsx @@ -13,6 +13,7 @@ import { useAnalyticsContext, useBackgroundServiceAPIContext, useTheme } from '@ import { PostHogAction } from '@providers/AnalyticsProvider/analyticsTracker'; import { useTranslation } from 'react-i18next'; import { BrowserViewSections } from '@lib/scripts/types'; +import { NotificationsBellContainer } from '@components/NotificationsCenter/NotificationsBellContainer'; export const MainHeader = (): React.ReactElement => { const { t } = useTranslation(); @@ -59,6 +60,7 @@ export const MainHeader = (): React.ReactElement => { label={t('expandPopup')} onClick={() => backgroundServices.handleOpenBrowser({ section: locationBrowserSection[location.pathname] })} /> + From dc4e87f0cea897e2f224494b5f5a34f1df6ce0ea Mon Sep 17 00:00:00 2001 From: Mircea Hasegan Date: Thu, 2 Oct 2025 19:02:11 +0200 Subject: [PATCH 2/3] refactor(extension): uniform notification center detection via hook --- .../NotificationsCenter/NotificationsBellContainer.tsx | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/apps/browser-extension-wallet/src/components/NotificationsCenter/NotificationsBellContainer.tsx b/apps/browser-extension-wallet/src/components/NotificationsCenter/NotificationsBellContainer.tsx index 77724a8d73..0e14bbb5e0 100644 --- a/apps/browser-extension-wallet/src/components/NotificationsCenter/NotificationsBellContainer.tsx +++ b/apps/browser-extension-wallet/src/components/NotificationsCenter/NotificationsBellContainer.tsx @@ -2,21 +2,20 @@ import React, { useState } from 'react'; import { useHistory } from 'react-router'; import { Dropdown } from 'antd'; -import { usePostHogClientContext } from '@providers/PostHogClientProvider'; -import { ExperimentName } from '@lib/scripts/types/feature-flags'; import { walletRoutePaths } from '@routes'; import { NotificationsBell } from './NotificationsBell'; import { NotificationsDropDown } from './NotificationsDropDown'; +import { useNotificationsCenterConfig } from '@hooks/useNotificationsCenterConfig'; export interface NotificationsCenterContainerProps { popupView?: boolean; } export const NotificationsBellContainer = ({ popupView }: NotificationsCenterContainerProps): React.ReactElement => { - const posthog = usePostHogClientContext(); const history = useHistory(); const [isOpen, setIsOpen] = useState(false); + const { isNotificationsCenterEnabled } = useNotificationsCenterConfig(); // TODO Connect with notifications center const [notifications, setNotifications] = useState([]); @@ -44,7 +43,7 @@ export const NotificationsBellContainer = ({ popupView }: NotificationsCenterCon }; return ( - posthog?.isFeatureFlagEnabled(ExperimentName.NOTIFICATIONS_CENTER) && ( + isNotificationsCenterEnabled && ( Date: Thu, 2 Oct 2025 19:04:52 +0200 Subject: [PATCH 3/3] fix(extension): reduce margin to allow notification icon to fit --- .../src/components/MainMenu/MainHeader.module.scss | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/apps/browser-extension-wallet/src/components/MainMenu/MainHeader.module.scss b/apps/browser-extension-wallet/src/components/MainMenu/MainHeader.module.scss index 9c5eb96723..d0b6ab0083 100644 --- a/apps/browser-extension-wallet/src/components/MainMenu/MainHeader.module.scss +++ b/apps/browser-extension-wallet/src/components/MainMenu/MainHeader.module.scss @@ -12,7 +12,7 @@ display: flex; align-items: center; justify-content: space-between; - margin: size_unit(1.75) size_unit(3); + margin: size_unit(1.75) size_unit(2.75); .linkLogo { display: flex; align-items: center;