Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commit0697308

Browse files
authored
fix: correctly display loading spinner (#16167)
Update the usages of the new spinner component to correctly set theloading prop
1 parent7f46e3b commit0697308

File tree

16 files changed

+19
-22
lines changed

16 files changed

+19
-22
lines changed

‎site/src/components/Dialogs/Dialog.tsx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ export const DialogActionButtons: FC<DialogActionButtonsProps> = ({
4848
data-testid="confirm-button"
4949
type="submit"
5050
>
51-
{confirmLoading&&<Spinnerloading/>}
51+
<Spinnerloading={confirmLoading}/>
5252
{confirmText}
5353
</Button>
5454
)}

‎site/src/pages/CreateTemplatePage/CreateTemplateForm.tsx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -357,7 +357,7 @@ export const CreateTemplateForm: FC<CreateTemplateFormProps> = (props) => {
357357
Cancel
358358
</Button>
359359
<Buttontype="submit"disabled={isSubmitting}>
360-
{isSubmitting&&<Spinner/>}
360+
<Spinnerloading={isSubmitting}/>
361361
{jobError ?"Retry" :"Save"}
362362
</Button>
363363
{logs&&(

‎site/src/pages/CreateTokenPage/CreateTokenForm.tsx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -152,7 +152,7 @@ export const CreateTokenForm: FC<CreateTokenFormProps> = ({
152152
Cancel
153153
</Button>
154154
<Buttontype="submit"disabled={isCreating}>
155-
{isCreating&&<Spinner/>}
155+
<Spinnerloading={isCreating}/>
156156
{creationFailed ?"Retry" :"Create token"}
157157
</Button>
158158
</FormFooter>

‎site/src/pages/CreateUserPage/CreateUserForm.tsx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -210,7 +210,7 @@ export const CreateUserForm: FC<
210210
Cancel
211211
</Button>
212212
<Buttontype="submit"disabled={isLoading}>
213-
{isLoading&&<Spinner/>}
213+
<Spinnerloading={isLoading}/>
214214
Save
215215
</Button>
216216
</FormFooter>

‎site/src/pages/CreateWorkspacePage/CreateWorkspacePageView.tsx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -322,7 +322,7 @@ export const CreateWorkspacePageView: FC<CreateWorkspacePageViewProps> = ({
322322
type="submit"
323323
disabled={creatingWorkspace||!hasAllRequiredExternalAuth}
324324
>
325-
{creatingWorkspace&&<Spinner/>}
325+
<Spinnerloading={creatingWorkspace}/>
326326
Create workspace
327327
</Button>
328328
</FormFooter>

‎site/src/pages/GroupsPage/SettingsGroupPageView.tsx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ const UpdateGroupForm: FC<UpdateGroupFormProps> = ({
113113
</Button>
114114

115115
<Buttontype="submit"disabled={isLoading}>
116-
{isLoading&&<Spinner/>}
116+
<Spinnerloading={isLoading}/>
117117
Save
118118
</Button>
119119
</FormFooter>

‎site/src/pages/ManagementSettingsPage/CustomRolesPage/CreateEditRolePageView.tsx‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,6 +99,7 @@ export const CreateEditRolePageView: FC<CreateEditRolePageViewProps> = ({
9999
form.handleSubmit();
100100
}}
101101
>
102+
<Spinnerloading={isLoading}/>
102103
{role!==undefined ?"Save" :"Create Role"}
103104
</Button>
104105
</div>
@@ -141,7 +142,7 @@ export const CreateEditRolePageView: FC<CreateEditRolePageViewProps> = ({
141142
</Button>
142143

143144
<Buttontype="submit"disabled={isLoading}>
144-
{isLoading&&<Spinner/>}
145+
<Spinnerloading={isLoading}/>
145146
{role ?"Save role" :"Create Role"}
146147
</Button>
147148
</FormFooter>

‎site/src/pages/ManagementSettingsPage/GroupsPage/CreateGroupPageView.tsx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ export const CreateGroupPageView: FC<CreateGroupPageViewProps> = ({
100100
</Button>
101101

102102
<Buttontype="submit"disabled={isLoading}>
103-
{isLoading&&<Spinner/>}
103+
<Spinnerloading={isLoading}/>
104104
Save
105105
</Button>
106106
</FormFooter>

‎site/src/pages/ManagementSettingsPage/GroupsPage/GroupSettingsPageView.tsx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,7 @@ const UpdateGroupForm: FC<UpdateGroupFormProps> = ({
124124
</Button>
125125

126126
<Buttontype="submit"disabled={isLoading}>
127-
{isLoading&&<Spinner/>}
127+
<Spinnerloading={isLoading}/>
128128
Save
129129
</Button>
130130
</FormFooter>

‎site/src/pages/ManagementSettingsPage/OrganizationSettingsPageView.tsx‎

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
importtype{Interpolation,Theme}from"@emotion/react";
2-
importButtonfrom"@mui/material/Button";
32
importTextFieldfrom"@mui/material/TextField";
43
import{isApiValidationError}from"api/errors";
54
importtype{
65
Organization,
76
UpdateOrganizationRequest,
87
}from"api/typesGenerated";
98
import{ErrorAlert}from"components/Alert/ErrorAlert";
9+
import{Button}from"components/Button/Button";
1010
import{DeleteDialog}from"components/Dialogs/DeleteDialog/DeleteDialog";
1111
import{
1212
FormFields,
@@ -119,7 +119,7 @@ export const OrganizationSettingsPageView: FC<
119119

120120
<FormFooter>
121121
<Buttontype="submit"disabled={form.isSubmitting}>
122-
{form.isSubmitting&&<Spinner/>}
122+
<Spinnerloading={form.isSubmitting}/>
123123
Save
124124
</Button>
125125
</FormFooter>
@@ -133,11 +133,7 @@ export const OrganizationSettingsPageView: FC<
133133
>
134134
<divcss={styles.dangerSettings}>
135135
<span>Deleting an organization is irreversible.</span>
136-
<Button
137-
css={styles.dangerButton}
138-
color="warning"
139-
onClick={()=>setIsDeleting(true)}
140-
>
136+
<Buttonvariant="destructive"onClick={()=>setIsDeleting(true)}>
141137
Delete this organization
142138
</Button>
143139
</div>

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp