From 386371baa3ee85e54c1f918c41297dc3c0681120 Mon Sep 17 00:00:00 2001
From: Hugh Rundle <hugh@hughrundle.net>
Date: Tue, 30 Nov 2021 12:42:52 +1100
Subject: [PATCH] improve a11y on notifications

Also change close link to a button.

Co-authored-by: Joachim <joachim.robert@protonmail.com>
---
 bookwyrm/templates/ostatus/error.html | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/bookwyrm/templates/ostatus/error.html b/bookwyrm/templates/ostatus/error.html
index 085fa3454..99a2a9acb 100644
--- a/bookwyrm/templates/ostatus/error.html
+++ b/bookwyrm/templates/ostatus/error.html
@@ -27,34 +27,34 @@
         </div>
     </div>
     {% elif error == 'ostatus_subscribe' %}
-    <div class="notification is-warning has-text-centered">
+    <div class="notification is-warning has-text-centered" role="status">
         <p>{% blocktrans %}Something went wrong trying to follow <strong>{{ account }}</strong>{% endblocktrans %}</p>
         <p>{% trans 'Check you have the correct username before trying again.' %}</p>
     </div>
     {% elif error == 'remote_subscribe' %}
-    <div class="notification is-warning has-text-centered">
+    <div class="notification is-warning has-text-centered" role="status">
         <p>{% blocktrans %}Something went wrong trying to follow from <strong>{{ account }}</strong>{% endblocktrans %}</p>
         <p>{% trans 'Check you have the correct username before trying again.' %}</p>
     </div>
     {% elif error == 'is_blocked' %}
-    <div class="notification is-danger has-text-centered">
+    <div class="notification is-danger has-text-centered" role="status">
         <p>{% blocktrans %}You have blocked <strong>{{ account }}</strong>{% endblocktrans %}</p>
     </div>
     {% elif error == 'has_blocked' %}
-    <div class="notification is-danger has-text-centered">
+    <div class="notification is-danger has-text-centered" role="status">
         <p>{% blocktrans %}<strong>{{ account }}</strong> has blocked you{% endblocktrans %}</p>
     </div>
     {% elif error == 'already_following' %}
-    <div class="notification is-success has-text-centered">
+    <div class="notification is-success has-text-centered" role="status">
         <p>{% blocktrans %}You are already following <strong>{{ account }}</strong>{% endblocktrans %}</p>
     </div>
     {% elif error == 'already_requested' %}
-    <div class="notification is-success has-text-centered">
+    <div class="notification is-success has-text-centered" role="status">
         <p>{% blocktrans %}You have already requested to follow <strong>{{ account }}</strong>{% endblocktrans %}</p>
     </div>
     {% endif %}
 </div>
 <div class="block is-pulled-right">
-    <a href="" onclick="closeWindow()">Close window</a>
+    <button type="button" onclick="closeWindow()">Close window</button>
 </div>
 {% endblock %}