Skip to content

Commit cb0532c

Browse files
Merge pull request #15015 from rhamilto/OCPBUGS-55633
OCPBUGS-55633: fix bug where Critical Alerts section cannot be collapsed
2 parents ea86bea + f381481 commit cb0532c

File tree

1 file changed

+4
-6
lines changed

1 file changed

+4
-6
lines changed

frontend/public/components/notification-drawer.tsx

+4-6
Original file line numberDiff line numberDiff line change
@@ -292,19 +292,17 @@ export const NotificationDrawer: React.FC<NotificationDrawerProps> = ({
292292

293293
const hasCriticalAlerts = criticalAlerts.length > 0;
294294
const hasNonCriticalAlerts = nonCriticalAlerts.length > 0;
295-
const [isAlertExpanded, toggleAlertExpanded] = React.useState<boolean>(hasCriticalAlerts);
296-
const [isNonCriticalAlertExpanded, toggleNonCriticalAlertExpanded] = React.useState<boolean>(
297-
true,
298-
);
299-
const [isClusterUpdateExpanded, toggleClusterUpdateExpanded] = React.useState<boolean>(true);
295+
const [isAlertExpanded, toggleAlertExpanded] = React.useState(hasCriticalAlerts);
296+
const [isNonCriticalAlertExpanded, toggleNonCriticalAlertExpanded] = React.useState(true);
297+
const [isClusterUpdateExpanded, toggleClusterUpdateExpanded] = React.useState(true);
300298
React.useEffect(() => {
301299
if (hasCriticalAlerts && isDrawerExpanded) {
302300
toggleAlertExpanded(true);
303301
}
304302
if (hasNonCriticalAlerts && isDrawerExpanded) {
305303
toggleNonCriticalAlertExpanded(true);
306304
}
307-
}, [hasCriticalAlerts, hasNonCriticalAlerts, isAlertExpanded, isDrawerExpanded]);
305+
}, [hasCriticalAlerts, hasNonCriticalAlerts, isDrawerExpanded]);
308306

309307
React.useEffect(() => {
310308
onDrawerChange();

0 commit comments

Comments
 (0)