Remove jQuery from the "quick submit" handler (#29200)
- Switched to plain JavaScript - Tested the quick submit functionality and it works as before # Demo using JavaScript without jQuery ![action](https://github.com/go-gitea/gitea/assets/20454870/abbd6c49-ad0f-4f95-b4ba-e969b85a46e8) --------- Signed-off-by: Yarden Shoham <git@yardenshoham.com>
This commit is contained in:
parent
2d8756a960
commit
d8d4b33b31
@ -1,5 +1,3 @@
|
|||||||
import $ from 'jquery';
|
|
||||||
|
|
||||||
export function handleGlobalEnterQuickSubmit(target) {
|
export function handleGlobalEnterQuickSubmit(target) {
|
||||||
const form = target.closest('form');
|
const form = target.closest('form');
|
||||||
if (form) {
|
if (form) {
|
||||||
@ -8,14 +6,9 @@ export function handleGlobalEnterQuickSubmit(target) {
|
|||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (form.classList.contains('form-fetch-action')) {
|
|
||||||
form.dispatchEvent(new SubmitEvent('submit', {bubbles: true, cancelable: true}));
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
|
|
||||||
// here use the event to trigger the submit event (instead of calling `submit()` method directly)
|
// here use the event to trigger the submit event (instead of calling `submit()` method directly)
|
||||||
// otherwise the `areYouSure` handler won't be executed, then there will be an annoying "confirm to leave" dialog
|
// otherwise the `areYouSure` handler won't be executed, then there will be an annoying "confirm to leave" dialog
|
||||||
$(form).trigger('submit');
|
form.dispatchEvent(new SubmitEvent('submit', {bubbles: true, cancelable: true}));
|
||||||
} else {
|
} else {
|
||||||
// if no form, then the editor is for an AJAX request, dispatch an event to the target, let the target's event handler to do the AJAX request.
|
// if no form, then the editor is for an AJAX request, dispatch an event to the target, let the target's event handler to do the AJAX request.
|
||||||
// the 'ce-' prefix means this is a CustomEvent
|
// the 'ce-' prefix means this is a CustomEvent
|
||||||
|
Loading…
Reference in New Issue
Block a user