From 11298514dd66dc347170115ec9eddd704d5e36c5 Mon Sep 17 00:00:00 2001
From: Pascal Wengerter <pascal@wengerter.info>
Date: Tue, 27 Apr 2021 10:56:57 +0100
Subject: [PATCH] Change from IDs to classes in quick action buttons

---
 changelog/unreleased/bugfix-quick-action-ids                | 6 ++++++
 .../src/components/FilesLists/QuickActions.vue              | 2 +-
 tests/acceptance/pageObjects/FilesPageElement/filesRow.js   | 2 +-
 3 files changed, 8 insertions(+), 2 deletions(-)
 create mode 100644 changelog/unreleased/bugfix-quick-action-ids

diff --git a/changelog/unreleased/bugfix-quick-action-ids b/changelog/unreleased/bugfix-quick-action-ids
new file mode 100644
index 00000000000..38fd25bf023
--- /dev/null
+++ b/changelog/unreleased/bugfix-quick-action-ids
@@ -0,0 +1,6 @@
+Bugfix: Change quick action from IDs to classes
+
+Quick action button IDs were repeated in every row of the file table, which isn't allowed in HTML (IDs must be unique per document).
+By changing to classes, this offense was resolved.
+
+https://github.com/owncloud/web/pull/5028
diff --git a/packages/web-app-files/src/components/FilesLists/QuickActions.vue b/packages/web-app-files/src/components/FilesLists/QuickActions.vue
index 088f8d3e071..a9d731cc8e0 100644
--- a/packages/web-app-files/src/components/FilesLists/QuickActions.vue
+++ b/packages/web-app-files/src/components/FilesLists/QuickActions.vue
@@ -2,12 +2,12 @@
   <div class="uk-flex">
     <oc-button
       v-for="action in filteredActions"
-      :id="`files-quick-action-${action.id}`"
       :key="action.label"
       :aria-label="action.label"
       :uk-tooltip="action.label"
       appearance="raw"
       class="oc-mr-xs"
+      :class="`files-quick-action-${action.id}`"
       @click.stop="action.handler({ item, client: $client, store: $store })"
     >
       <oc-icon :name="action.icon" class="uk-flex" />
diff --git a/tests/acceptance/pageObjects/FilesPageElement/filesRow.js b/tests/acceptance/pageObjects/FilesPageElement/filesRow.js
index ee584d6ad12..d59d1b90acf 100644
--- a/tests/acceptance/pageObjects/FilesPageElement/filesRow.js
+++ b/tests/acceptance/pageObjects/FilesPageElement/filesRow.js
@@ -50,7 +50,7 @@ module.exports = {
       locateStrategy: 'xpath'
     },
     quickAction: {
-      selector: '//button[@id="files-quick-action-%s"]',
+      selector: '//button[@class="files-quick-action-%s"]',
       locateStrategy: 'xpath'
     }
   }