Replace autosave mixin with utility functions
The following discussion from !25563 (merged) should be addressed:
-
@winh started a discussion: (+1 comment) I have created the utility functions above because the existing
app/assets/javascripts/autosave.js
/app/assets/javascripts/notes/mixins/autosave.js
add event handlers to elements inside the Vue template outside of the Vue component using jQuery—which is a bad idea
There are two components that should be updated as well:
$ grep -lr 'mixins/autosave' {ee/,}app/assets/javascripts
app/assets/javascripts/notes/components/note_body.vue
app/assets/javascripts/diffs/components/diff_line_note_form.vue