Skip to content

Commit 23795f2

Browse files
authored
fix(ui): fixed sidenav active state (#19875)
1 parent 62e717f commit 23795f2

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

src/sentry/static/sentry/app/views/settings/components/settingsLayout.tsx

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -168,8 +168,6 @@ const SidebarWrapper = styled('div')<{isVisible: boolean; offsetTop: number}>`
168168
width: ${p => p.theme.settings.sidebarWidth};
169169
background: ${p => p.theme.white};
170170
border-right: 1px solid ${p => p.theme.borderLight};
171-
padding: ${space(4)};
172-
padding-right: ${space(2)};
173171
174172
@media (max-width: ${p => p.theme.breakpoints[0]}) {
175173
display: ${p => (p.isVisible ? 'block' : 'none')};

src/sentry/static/sentry/app/views/settings/components/settingsNavigation.tsx

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ import React from 'react';
22
import * as Sentry from '@sentry/react';
33
import styled from '@emotion/styled';
44

5+
import space from 'app/styles/space';
56
import SettingsNavigationGroup from 'app/views/settings/components/settingsNavigationGroup';
67
import {NavigationSection, NavigationProps} from 'app/views/settings/types';
78

@@ -58,11 +59,13 @@ class SettingsNavigation extends React.Component<Props> {
5859
const PositionStickyWrapper = styled('div')`
5960
@media (min-width: ${p => p.theme.breakpoints[0]}) {
6061
position: sticky;
61-
top: 100px;
62+
top: 70px;
6263
overflow: scroll;
63-
height: calc(100vh - 98px);
64+
height: calc(100vh - 70px);
6465
-ms-overflow-style: none;
6566
scrollbar-width: none;
67+
padding: ${space(4)};
68+
padding-right: ${space(2)};
6669
6770
&::-webkit-scrollbar {
6871
display: none;

0 commit comments

Comments
 (0)