diff options
| author | wxiaoguang <wxiaoguang@gmail.com> | 2023-06-13 20:10:10 +0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-06-13 12:10:10 +0000 |
| commit | 6bbccdd177bcbac600e3019062c96339a121851a (patch) | |
| tree | 92d53bbdd976b6b8657b10ccda910a08d094dc81 /templates/repo/settings/options.tmpl | |
| parent | a51b115b0ad755562c71f6a382326c59868dfb96 (diff) | |
| download | gitea-6bbccdd177bcbac600e3019062c96339a121851a.tar.gz gitea-6bbccdd177bcbac600e3019062c96339a121851a.zip | |
Improve AJAX link and modal confirm dialog (#25210)
Clarify the "link-action" behavior:
> // A "link-action" can post AJAX request to its "data-url"
> // Then the browser is redirect to: the "redirect" in response, or
"data-redirect" attribute, or current URL by reloading.
And enhance the "link-action" to support showing a modal dialog for
confirm. A similar general approach could also help PRs like
https://github.com/go-gitea/gitea/pull/22344#discussion_r1062883436
> // If the "link-action" has "data-modal-confirm(-html)" attribute, a
confirm modal dialog will be shown before taking action.
And a lot of duplicate code can be removed now. A good framework design
can help to avoid code copying&pasting.
---------
Co-authored-by: silverwind <me@silverwind.io>
Diffstat (limited to 'templates/repo/settings/options.tmpl')
| -rw-r--r-- | templates/repo/settings/options.tmpl | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/templates/repo/settings/options.tmpl b/templates/repo/settings/options.tmpl index 1e940c581b..a1fe1ad5af 100644 --- a/templates/repo/settings/options.tmpl +++ b/templates/repo/settings/options.tmpl @@ -61,7 +61,7 @@ <div class="field"> <button class="ui green button">{{$.locale.Tr "settings.update_avatar"}}</button> - <a class="ui red button delete-post" data-request-url="{{.Link}}/avatar/delete" data-done-url="{{.Link}}">{{$.locale.Tr "settings.delete_current_avatar"}}</a> + <button class="ui red button link-action" data-url="{{.Link}}/avatar/delete" data-redirect="{{.Link}}">{{$.locale.Tr "settings.delete_current_avatar"}}</button> </div> </form> |