Skip to content

Commit d77ee89

Browse files
committed
fix: ux nits and simplify
1 parent d314847 commit d77ee89

File tree

1 file changed

+14
-22
lines changed

1 file changed

+14
-22
lines changed

src/layouts/TopBarNotify.tsx

Lines changed: 14 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -47,44 +47,36 @@ export default function TopBarNotify({ campaigns }: TopBarNotifyProps) {
4747
const mobileDrawerOpen = useRootStore((state) => state.mobileDrawerOpen);
4848

4949
const getCurrentCampaign = (): CampaignConfig | null => {
50-
const chainIds = Object.keys(campaigns).map(Number);
51-
const firstChainId = chainIds[0];
52-
return firstChainId ? campaigns[firstChainId] || null : null;
50+
return campaigns[currentChainId] || null;
5351
};
5452

5553
const currentCampaign = getCurrentCampaign();
56-
const campaignChainId = currentCampaign
57-
? Object.keys(campaigns)
58-
.map(Number)
59-
.find((chainId) => campaigns[chainId] === currentCampaign) || currentChainId
60-
: currentChainId;
54+
const campaignChainId = currentChainId;
6155

62-
const [showWarning, setShowWarning] = useState(() => {
63-
if (!currentCampaign) return false;
64-
65-
const storedBannerVersion = localStorage.getItem(`bannerVersion_${campaignChainId}`);
66-
const warningBarOpen = localStorage.getItem(`warningBarOpen_${campaignChainId}`);
67-
68-
if (storedBannerVersion !== currentCampaign.bannerVersion) {
69-
return true;
70-
}
71-
72-
return warningBarOpen !== 'false';
73-
});
56+
const [showWarning, setShowWarning] = useState(false);
7457

7558
const [slideIn, setSlideIn] = useState(false);
7659

7760
useEffect(() => {
78-
if (!currentCampaign) return;
61+
if (!currentCampaign) {
62+
setShowWarning(false);
63+
setSlideIn(false);
64+
return;
65+
}
7966

8067
const storedBannerVersion = localStorage.getItem(`bannerVersion_${campaignChainId}`);
68+
const warningBarOpen = localStorage.getItem(`warningBarOpen_${campaignChainId}`);
8169

70+
// Check if this is a new banner version for this chain
8271
if (storedBannerVersion !== currentCampaign.bannerVersion) {
8372
localStorage.setItem(`bannerVersion_${campaignChainId}`, currentCampaign.bannerVersion);
8473
localStorage.setItem(`warningBarOpen_${campaignChainId}`, 'true');
8574
setShowWarning(true);
75+
} else {
76+
// Use stored preference for this chain
77+
setShowWarning(warningBarOpen !== 'false');
8678
}
87-
}, [currentCampaign, campaignChainId]);
79+
}, [currentCampaign, campaignChainId, currentChainId]);
8880

8981
useEffect(() => {
9082
if (showWarning) {

0 commit comments

Comments
 (0)