Update action, fix linter

Author: fsbraun

.github/workflows/frontend.yml

@@ -27,18 +27,17 @@ jobs:
     - name: Install dependencies
       run: npm ci
 
-    - name: Install Playwright browsers
-      run: npx playwright install --with-deps
+    - name: Install Playwright Chromium browser
+      run: npx playwright install --with-deps chromium
 
-    - name: Run linting
+    - name: Run linter
       run: npx gulp lint
 
     - name: Run E2E tests with coverage
-      run: npm run e2e
+      run: |
+        npm run e2e
+        npm run e2e:coverage:report
 
-    - name: Generate coverage report
-      if: always()
-      run: npm run e2e:coverage:report
 
     - name: Upload coverage reports
       if: always()

djangocms_versioning/static/djangocms_versioning/js/compare.js

@@ -6,16 +6,16 @@ function bindHandlers(submitButton) {
 }
 
 window.addEventListener('load', function() {
-  document.querySelector('select[name=action]').addEventListener('change', function() {
-    let selectVal = this.options[this.selectedIndex].getAttribute("value");
-    const actionsForm = document.getElementById('changelist-form')
-    const submitButton = actionsForm.querySelector('[type="submit"]')
+    document.querySelector('select[name=action]').addEventListener('change', function() {
+        let selectVal = this.options[this.selectedIndex].getAttribute('value');
+        const actionsForm = document.getElementById('changelist-form');
+        const submitButton = actionsForm.querySelector('[type="submit"]');
 
         if (selectVal === 'compare_versions') {
         // Setting the target to blank to ensure the compare view is opened in a new tab
-        actionsForm.setAttribute('target', '_blank');
-        bindHandlers(submitButton);
-    }else {
+            actionsForm.setAttribute('target', '_blank');
+            bindHandlers(submitButton);
+        } else {
         // If the user deselect the compare version action, the "target=_blank" is removed
             actionsForm.removeAttribute('target');
         }

package.json

@@ -50,10 +50,7 @@
     "srcdoc-polyfill": "^1.0.0"
   },
   "browserslist": [
-    "last 2 Chrome versions",
-    "last 2 Firefox versions",
-    "last 2 Safari versions",
-    "last 2 Edge versions"
+    "last 2 Chrome versions"
   ],
   "nyc": {
     "reporter": ["lcov", "text-summary"],

playwright.config.js

@@ -13,4 +13,10 @@ module.exports = defineConfig({
     baseURL: 'http://localhost:8089',
     headless: true,
   },
+  projects: [
+    {
+      name: 'chromium',
+      use: { browserName: 'chromium' },
+    },
+  ],
 });