Skip to content

Commit 3c3c5cc

Browse files
authored
refactor: remove nprogress loading indicator (#1790)
Signed-off-by: Adam Setch <[email protected]>
1 parent abbb4e6 commit 3c3c5cc

File tree

7 files changed

+1
-111
lines changed

7 files changed

+1
-111
lines changed

Diff for: package.json

-2
Original file line numberDiff line numberDiff line change
@@ -152,7 +152,6 @@
152152
"@testing-library/react": "16.2.0",
153153
"@types/jest": "29.5.14",
154154
"@types/node": "22.12.0",
155-
"@types/nprogress": "0.2.3",
156155
"@types/react": "19.0.8",
157156
"@types/react-dom": "19.0.3",
158157
"@types/react-router-dom": "5.3.3",
@@ -174,7 +173,6 @@
174173
"jest-environment-jsdom": "29.7.0",
175174
"mini-css-extract-plugin": "2.9.2",
176175
"nock": "13.5.6",
177-
"nprogress": "0.2.0",
178176
"postcss": "8.5.1",
179177
"postcss-loader": "8.1.1",
180178
"rimraf": "6.0.1",

Diff for: pnpm-lock.yaml

-16
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Diff for: src/renderer/App.tsx

-2
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ import {
99

1010
import { BaseStyles, Box, ThemeProvider } from '@primer/react';
1111

12-
import { Loading } from './components/Loading';
1312
import { Sidebar } from './components/Sidebar';
1413
import { AppContext, AppProvider } from './context/App';
1514
import { AccountsRoute } from './routes/Accounts';
@@ -40,7 +39,6 @@ export const App = () => {
4039
<AppProvider>
4140
<Router>
4241
<Box className="flex flex-col min-h-screen overflow-x-hidden overflow-y-auto pl-sidebar bg-gitify-background">
43-
<Loading />
4442
<Sidebar />
4543
<Routes>
4644
<Route

Diff for: src/renderer/components/Loading.test.tsx

-52
This file was deleted.

Diff for: src/renderer/components/Loading.tsx

-29
This file was deleted.

Diff for: src/renderer/components/fields/RadioGroup.test.tsx

+1-9
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { fireEvent, render, screen } from '@testing-library/react';
1+
import { render } from '@testing-library/react';
22
import { type IRadioGroup, RadioGroup } from './RadioGroup';
33

44
describe('renderer/components/fields/RadioGroup.tsx', () => {
@@ -24,12 +24,4 @@ describe('renderer/components/fields/RadioGroup.tsx', () => {
2424
const tree = render(<RadioGroup {...mockProps} />);
2525
expect(tree).toMatchSnapshot();
2626
});
27-
28-
it('should check that NProgress is getting called in getDerivedStateFromProps (loading)', () => {
29-
render(<RadioGroup {...props} />);
30-
31-
fireEvent.click(screen.getByLabelText('Value 1'));
32-
33-
expect(props.onChange).toHaveBeenCalledTimes(1);
34-
});
3527
});

Diff for: src/renderer/index.tsx

-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
import { createRoot } from 'react-dom/client';
22

3-
import 'nprogress/nprogress.css';
43
import { App } from './App';
54

65
const container = document.getElementById('root');

0 commit comments

Comments
 (0)