Skip to content

Commit e88f9f5

Browse files
committed
feat: shrink sidebar width
Signed-off-by: Adam Setch <[email protected]>
1 parent 8f2b5c0 commit e88f9f5

File tree

6 files changed

+13
-7
lines changed

6 files changed

+13
-7
lines changed

src/renderer/App.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ export const App = () => {
3737
<BaseStyles>
3838
<AppProvider>
3939
<Router>
40-
<div className="flex h-full overflow-y-auto flex-col pl-14">
40+
<div className="flex h-full overflow-y-auto flex-col pl-10">
4141
<Loading />
4242
<Sidebar />
4343
<Routes>

src/renderer/components/Sidebar.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ export const Sidebar: FC = () => {
7272
}, [settings]);
7373

7474
return (
75-
<div className="fixed left-14 -ml-14 flex h-full w-14 flex-col overflow-y-auto bg-gray-sidebar">
75+
<div className="fixed left-10 -ml-10 flex h-full w-10 flex-col overflow-y-auto bg-gray-sidebar">
7676
<div className="flex flex-1 flex-col">
7777
<Stack
7878
direction="vertical"

src/renderer/components/__snapshots__/Sidebar.test.tsx.snap

Lines changed: 4 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/renderer/routes/Accounts.test.tsx

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -257,6 +257,7 @@ describe('renderer/routes/Accounts.tsx', () => {
257257
);
258258
});
259259

260+
fireEvent.click(screen.getByTestId('account-add-new'));
260261
fireEvent.click(screen.getByTestId('account-add-github'));
261262

262263
expect(mockLoginWithGitHubApp).toHaveBeenCalled();
@@ -278,6 +279,7 @@ describe('renderer/routes/Accounts.tsx', () => {
278279
);
279280
});
280281

282+
fireEvent.click(screen.getByTestId('account-add-new'));
281283
fireEvent.click(screen.getByTestId('account-add-pat'));
282284

283285
expect(mockNavigate).toHaveBeenNthCalledWith(
@@ -305,6 +307,7 @@ describe('renderer/routes/Accounts.tsx', () => {
305307
);
306308
});
307309

310+
fireEvent.click(screen.getByTestId('account-add-new'));
308311
fireEvent.click(screen.getByTestId('account-add-oauth-app'));
309312

310313
expect(mockNavigate).toHaveBeenNthCalledWith(1, '/login-oauth-app', {

src/renderer/routes/Accounts.tsx

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -168,7 +168,9 @@ export const AccountsRoute: FC = () => {
168168
<div className="flex items-center justify-end bg-gray-200 px-8 py-1 text-sm dark:bg-gray-darker">
169169
<ActionMenu>
170170
<ActionMenu.Anchor>
171-
<Button leadingVisual={PersonAddIcon}>Add new account</Button>
171+
<Button leadingVisual={PersonAddIcon} data-testid="account-add-new">
172+
Add new account
173+
</Button>
172174
</ActionMenu.Anchor>
173175

174176
<ActionMenu.Overlay width="medium">

src/renderer/routes/__snapshots__/Accounts.test.tsx.snap

Lines changed: 1 addition & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)