Before:
<img width="364" alt="Screen Shot 2023-06-20 at 11 59 11"
src="https://github.com/go-gitea/gitea/assets/
17645053/
ad284b7e-8d21-43be-b178-
bbcfd37cb5bd">
Might trigger many posts when keep clicking the buttons above.
<img width="448" alt="Screen Shot 2023-06-20 at 11 52 28"
src="https://github.com/go-gitea/gitea/assets/
17645053/
a60aa6ac-af74-45e4-b13a-
512b436b81b0">
<img width="678" alt="Screen Shot 2023-06-20 at 11 52 37"
src="https://github.com/go-gitea/gitea/assets/
17645053/
d6662700-3643-4cc7-a2ec-
64e1c0f5fbdb">
After (PR sidebar, Same for issue):
https://github.com/go-gitea/gitea/assets/
17645053/
9df3ad1f-e29c-439b-8bde-
e6b917d63cc6
For delete, it is using `base/modal_actions_confirm` subtemplate, and we
might need another general solution for this (maybe add another
attribute to the subtemplate or something)
---------
Co-authored-by: silverwind <me@silverwind.io>
Co-authored-by: Giteabot <teabot@gitea.io>
Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
}
if issue.IsLocked {
- ctx.Flash.Error(ctx.Tr("repo.issues.lock_duplicate"))
- ctx.Redirect(issue.Link())
+ ctx.JSONError(ctx.Tr("repo.issues.lock_duplicate"))
return
}
if !form.HasValidReason() {
- ctx.Flash.Error(ctx.Tr("repo.issues.lock.unknown_reason"))
- ctx.Redirect(issue.Link())
+ ctx.JSONError(ctx.Tr("repo.issues.lock.unknown_reason"))
return
}
return
}
- ctx.Redirect(issue.Link())
+ ctx.JSONRedirect(issue.Link())
}
// UnlockIssue unlocks a previously locked issue.
}
if !issue.IsLocked {
- ctx.Flash.Error(ctx.Tr("repo.issues.unlock_error"))
- ctx.Redirect(issue.Link())
+ ctx.JSONError(ctx.Tr("repo.issues.unlock_error"))
return
}
return
}
- ctx.Redirect(issue.Link())
+ ctx.JSONRedirect(issue.Link())
}
return
}
- ctx.Redirect(issue.Link())
+ ctx.JSONRedirect(issue.Link())
}
// IssueUnpin unpins a Issue
<div class="ui divider"></div>
{{if or .PinEnabled .Issue.IsPinned}}
- <form class="gt-mt-2" method="post" {{if $.NewPinAllowed}}action="{{.Issue.Link}}/pin"{{else}}data-tooltip-content="{{.locale.Tr "repo.issues.max_pinned"}}"{{end}}>
+ <form class="gt-mt-2 form-fetch-action single-button-form" method="post" {{if $.NewPinAllowed}}action="{{.Issue.Link}}/pin"{{else}}data-tooltip-content="{{.locale.Tr "repo.issues.max_pinned"}}"{{end}}>
{{$.CsrfTokenHtml}}
<button class="fluid ui button {{if not $.NewPinAllowed}}disabled{{end}}">
{{if not .Issue.IsPinned}}
{{end}}
</div>
- <form class="ui form" action="{{.Issue.Link}}{{if .Issue.IsLocked}}/unlock{{else}}/lock{{end}}"
+ <form class="ui form form-fetch-action" action="{{.Issue.Link}}{{if .Issue.IsLocked}}/unlock{{else}}/lock{{end}}"
method="post">
{{.CsrfTokenHtml}}
content: "";
position: absolute;
display: block;
- height: 4rem;
- aspect-ratio: 1 / 1;
left: 50%;
top: 50%;
- max-height: 100%;
- max-width: 100%;
+ height: min(4em, 66.6%);
+ aspect-ratio: 1;
transform: translate(-50%, -50%);
animation: isloadingspin 500ms infinite linear;
border-width: 4px;
border-width: 2px;
}
+/* for single form button, the loading state should be on the button, but not go semi-transparent, just replace the text on the button with the loader. */
+form.single-button-form.is-loading > * {
+ opacity: 1;
+}
+
+form.single-button-form.is-loading .button {
+ color: transparent;
+}
+
.markup pre.is-loading,
.editor-loading.is-loading,
.pdf-content.is-loading {