From 6b75985f9610f6237e7783e80d7febc2a3bf9b05 Mon Sep 17 00:00:00 2001
From: Simon Ser <contact@emersion.fr>
Date: Sun, 6 Jun 2021 12:05:57 +0200
Subject: [PATCH] Make <summary> easier to click on Firefox Mobile
Add role=button to mark the element as a touch target.
Closes: https://todo.sr.ht/~emersion/gamja/80
---
components/connect-form.js | 2 +-
components/network-form.js | 2 +-
style.css | 1 -
3 files changed, 2 insertions(+), 3 deletions(-)
diff --git a/components/connect-form.js b/components/connect-form.js
index ae80c64..4deb947 100644
--- a/components/connect-form.js
+++ b/components/connect-form.js
@@ -121,7 +121,7 @@ export default class ConnectForm extends Component {
<br/><br/>
<details>
- <summary>Advanced options</summary>
+ <summary role="button">Advanced options</summary>
<br/>
diff --git a/components/network-form.js b/components/network-form.js
index bae5089..a3f9392 100644
--- a/components/network-form.js
+++ b/components/network-form.js
@@ -76,7 +76,7 @@ export default class NetworkForm extends Component {
<br/><br/>
<details>
- <summary>Advanced options</summary>
+ <summary role="button">Advanced options</summary>
<br/>
diff --git a/style.css b/style.css
index 753a502..bff718b 100644
--- a/style.css
+++ b/style.css
@@ -62,7 +62,6 @@ button {
background: var(--button-background);
transition: background 0.25s linear;
color: var(--button-color);
- cursor: pointer;
padding: 0.25rem 0.5rem;
border-style: solid;
border-color: var(--button-border);