Skip to content

Commit 566d427

Browse files
committed
Merge branches 'w/133.0/improvement/navbar-style-improvements' and 'q/w/4755/132.0/improvement/navbar-style-improvements' into tmp/octopus/q/133.0
3 parents af9ee74 + 8e24310 + fb4fd8c commit 566d427

File tree

3 files changed

+8
-6
lines changed

3 files changed

+8
-6
lines changed

shell-ui/src/navbar/NavBar.tsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ import type { Link as TypeLink } from './navbarHooks';
3434
import { useNavbar } from './navbarHooks';
3535

3636
const Logo = styled.img`
37-
height: 2.143rem;
37+
height: 1.5rem;
3838
`;
3939
export const LoadingNavbar = ({ logo }: { logo: string }) => (
4040
<CoreUINavbar
@@ -403,6 +403,7 @@ export const Navbar = ({
403403
rightTabs.unshift({
404404
render: () => (
405405
<Button
406+
style={{ backgroundColor: 'transparent' }}
406407
onClick={() => {
407408
setThemeMode(switchThemeTo);
408409
}}

shell-ui/src/navbar/NotificationCenter.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ const NotificationCenter = () => {
113113
const navigate = useShellHistory();
114114
return (
115115
<div
116-
style={{ position: 'relative' }}
116+
style={{ position: 'relative'}}
117117
{...getToggleButtonProps({
118118
'aria-label': 'Notification Center',
119119
ref: (e: HTMLDivElement) => {
@@ -127,7 +127,7 @@ const NotificationCenter = () => {
127127
})}
128128
>
129129
<Button
130-
style={isOpen ? { background: theme?.highlight } : {}}
130+
style={isOpen ? { background: theme?.highlight } : { backgroundColor: 'transparent' }}
131131
icon={
132132
<span className="fa-layers fa-fw">
133133
<Icon

ui/src/containers/AlertPage.tsx

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ const SecondaryTitle = styled.div`
4444
justify-content: center;
4545
font-size: ${fontSize.base};
4646
width: 250px;
47-
color: ${(props) => props.theme.textSecondary};
47+
color: ${(props) => props.theme.textPrimary};
4848
`;
4949
const TertiaryTitle = styled.div`
5050
display: flex;
@@ -184,8 +184,9 @@ const ActiveAlertTab = React.memo(
184184
startsAt: (row1, row2) => {
185185
const a = row1?.values?.startsAt;
186186
const b = row2.values?.startsAt;
187-
// @ts-expect-error - FIXME when you are working on it
188-
return new Date(a) - new Date(b);
187+
const dateA = new Date(a).getTime();
188+
const dateB = new Date(b).getTime();
189+
return dateA - dateB;
189190
},
190191
};
191192
}, []);

0 commit comments

Comments
 (0)