Update JS deps

.github/workflows/test-unit.yml

@@ -297,6 +297,7 @@ jobs:
         run: |
           mv public public.orig
           mkdir public public/css public/external
+          cp public.orig/.gitattributes public
           cp public.orig/.gitignore public
           cp public.orig/logo.png public
           cp public.orig/css/agileui.less public/css

js/src/plugins/scroll.plugin.js

@@ -96,7 +96,7 @@ export default class scroll extends atkPlugin {
     /**
      * Check if scrolling require adding content.
      *
-     * @param e // event
+     * @param e event
      */
     observe(e) {
         const borderTopWidth = parseInt(this.$el.css('borderTopWidth'), 10);

js/src/services/api.service.js

@@ -29,8 +29,8 @@ class ApiService {
      * By passig the jQuery reference, $ var use by code that need to be eval
      * will work just fine, even if $ is not assign globally.
      *
-     * @param code // javascript to be eval.
-     * @param $    // reference to jQuery.
+     * @param code javascript to be eval.
+     * @param $    reference to jQuery.
      */
     evalResponse(code, $) { // eslint-disable-line no-shadow
         eval(code); // eslint-disable-line no-eval

public/.gitattributes

@@ -0,0 +1,3 @@
+# workaround for vue
+# https://github.com/vuejs/vue/issues/12819
+/js/* -text -filter

public/.gitignore

@@ -1,4 +1,5 @@
 /*
+!/.gitattributes
 !/.gitignore
 !/logo.png
 !/*/

public/js/atk-vue-inline-edit.js

@@ -106,7 +106,9 @@ const template = `
     clearError: function () {
       this.hasError = false;
     },
-    flashError: function (count = 4) {
+    flashError: function () {
+      let count = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : 4;
+
       if (count === 0) {
         this.hasError = false;
         return;