fix(a11y): Copy menu entries should be implemented as buttons

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
nextcloud · Jun 21, 2023 · ea99fca · ea99fca
1 parent dad002f
commit ea99fca
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/Components/LogTable.css b/src/Components/LogTable.css
@@ -115,7 +115,7 @@
 		margin: 0;
 		overflow: visible;
 
-		button {
+		> button {
 			position: relative;
 			opacity: 0;
 			transition: opacity 0.5s;

diff --git a/src/Components/LogTable.js b/src/Components/LogTable.js
@@ -89,16 +89,16 @@ export class LogTable extends Component {
 									className={style.copyMenu + ' popovermenu bubble open menu'}>
 									<ul>
 										<li>
-											<a className="menuitem icon icon-clippy"
+											<button className="menuitem icon icon-clippy"
 											   onClick={() => copyEntry(true)}>
 												{t('logreader', 'Copy raw')}
-											</a>
+											</button>
 										</li>
 										<li>
-											<a className="menuitem icon icon-clippy"
+											<button className="menuitem icon icon-clippy"
 											   onClick={() => copyEntry(false)}>
 												{t('logreader', 'Copy formatted')}
-											</a>
+											</button>
 										</li>
 									</ul>
 								</div> :