Skip to content

Commit 21db7da

Browse files
committed
feat: Update @vscode/codicons to v0.0.33
They renamed circle-large-outline to circle-large for consistency. Update adds several new icons. I've fixed our usage to match for the renamed icon.
1 parent 769d753 commit 21db7da

6 files changed

Lines changed: 16 additions & 16 deletions

File tree

package-lock.json

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

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@
101101
"@types/react-window": "^1.8.5",
102102
"@types/shell-quote": "^1.7.1",
103103
"@types/shortid": "0.0.29",
104-
"@vscode/codicons": "0.0.32",
104+
"@vscode/codicons": "0.0.33",
105105
"chokidar-cli": "^2.1.0",
106106
"cross-env": "^7.0.2",
107107
"eslint": "^8.29.0",

packages/components/src/LoadingSpinner.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import React from 'react';
22
import classNames from 'classnames';
33
import { FontAwesomeIcon } from '@fortawesome/react-fontawesome';
4-
import { vsCircleLargeOutline, vsLoading } from '@deephaven/icons';
4+
import { vsCircleLarge, vsLoading } from '@deephaven/icons';
55
import './LoadingSpinner.scss';
66

77
type LoadingSpinnerProps = {
@@ -18,7 +18,7 @@ function LoadingSpinner({
1818
className={classNames('loading-spinner fa-layers', className)}
1919
data-testid={dataTestId}
2020
>
21-
<FontAwesomeIcon icon={vsCircleLargeOutline} className="text-white-50" />
21+
<FontAwesomeIcon icon={vsCircleLarge} className="text-white-50" />
2222
<FontAwesomeIcon icon={vsLoading} className="text-primary" spin />
2323
</div>
2424
);

packages/components/src/context-actions/__snapshots__/ContextActions.test.tsx.snap

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,8 +37,8 @@ exports[`renders a menu from a promise returned from a function 1`] = `
3737
>
3838
<svg
3939
aria-hidden="true"
40-
className="svg-inline--fa fa-circle-large-outline text-white-50"
41-
data-icon="circle-large-outline"
40+
className="svg-inline--fa fa-circle-large text-white-50"
41+
data-icon="circle-large"
4242
data-prefix="vs"
4343
focusable="false"
4444
role="img"
@@ -206,8 +206,8 @@ exports[`renders an empty menu for a rejected promise 1`] = `
206206
>
207207
<svg
208208
aria-hidden="true"
209-
className="svg-inline--fa fa-circle-large-outline text-white-50"
210-
data-icon="circle-large-outline"
209+
className="svg-inline--fa fa-circle-large text-white-50"
210+
data-icon="circle-large"
211211
data-prefix="vs"
212212
focusable="false"
213213
role="img"

packages/dashboard-core-plugins/src/panels/ChartPanel.test.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ function callErrorFunction() {
102102

103103
function expectLoading(container) {
104104
expect(
105-
container.querySelector("[data-icon='circle-large-outline']")
105+
container.querySelector("[data-icon='circle-large']")
106106
).toBeInTheDocument();
107107
expect(container.querySelector("[data-icon='loading']")).toBeInTheDocument();
108108
}

packages/dashboard-core-plugins/src/panels/IrisGridPanel.test.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@ function makeIrisGridPanelWrapper(
8181
async function expectLoading(container) {
8282
await waitFor(() =>
8383
expect(
84-
container.querySelector("[data-icon='circle-large-outline']")
84+
container.querySelector("[data-icon='circle-large']")
8585
).toBeInTheDocument()
8686
);
8787
expect(container.querySelector("[data-icon='loading']")).toBeInTheDocument();

0 commit comments

Comments
 (0)