From b2e47e34e543d8585542f5a9a53a399f2a612f46 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bj=C3=B6rn=20Nilsved?= <nilsved@gmail.com>
Date: Fri, 1 May 2020 16:38:09 +0200
Subject: [PATCH] Use role="list" to keep list-semantics in safari

https://www.scottohara.me/blog/2019/01/12/lists-and-safari.html#a-new-alternative

Closes #142
---
 resources/views/forms/partials/errors.blade.php        | 2 +-
 resources/views/widgets/crumbtrail.blade.php           | 2 +-
 resources/views/widgets/itemHistory.blade.php          | 2 +-
 resources/views/widgets/menu.blade.php                 | 4 ++--
 resources/views/widgets/message.blade.php              | 2 +-
 resources/views/widgets/personalRecentVisits.blade.php | 2 +-
 resources/views/widgets/teamRecentVisits.blade.php     | 2 +-
 tests/Feature/UserlandControllerTest.php               | 2 +-
 8 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/resources/views/forms/partials/errors.blade.php b/resources/views/forms/partials/errors.blade.php
index 7eeea903..30c7f073 100644
--- a/resources/views/forms/partials/errors.blade.php
+++ b/resources/views/forms/partials/errors.blade.php
@@ -1,5 +1,5 @@
 @if($errors->hasAny($errorsKeys = $errorsKeys ?? $name))
-  <ul id="{{ $errorsId }}">
+  <ul id="{{ $errorsId }}" role="list">
     @foreach(\Illuminate\Support\Arr::wrap($errorsKeys) as $key)
       @foreach(\Illuminate\Support\Arr::flatten($errors->get($key)) as $message)
         <li>{{ $message }}</li>
diff --git a/resources/views/widgets/crumbtrail.blade.php b/resources/views/widgets/crumbtrail.blade.php
index fc660a51..ceecbf31 100644
--- a/resources/views/widgets/crumbtrail.blade.php
+++ b/resources/views/widgets/crumbtrail.blade.php
@@ -1,5 +1,5 @@
 <nav aria-label="{{ __('Crumb trail') }}" data-kontour-widget="crumbtrail">
-  <ol>
+  <ol role="list">
     @foreach($links as $link)
       <li{!! url()->full() == $link->getUrl() ? ' aria-current="true"' : '' !!}>{{ $link }}</li>
     @endforeach
diff --git a/resources/views/widgets/itemHistory.blade.php b/resources/views/widgets/itemHistory.blade.php
index 6010f3ed..099e039a 100644
--- a/resources/views/widgets/itemHistory.blade.php
+++ b/resources/views/widgets/itemHistory.blade.php
@@ -1,6 +1,6 @@
 <section data-kontour-widget="itemHistory">
   <header>{{ trans('kontour::widgets.itemHistory.title') }}</header>
-  <ul>
+  <ul role="list">
   @foreach($entries as $entry)
     <li lang="en" data-kontour-entry-action="{{ $entry['action'] }}"@if($entry['user']) data-kontour-username="{{ $entry['user']->getDisplayName() }}"@endif>
         <span>{{ ucfirst($entry['action']) }}</span>
diff --git a/resources/views/widgets/menu.blade.php b/resources/views/widgets/menu.blade.php
index 43778688..6aba405d 100644
--- a/resources/views/widgets/menu.blade.php
+++ b/resources/views/widgets/menu.blade.php
@@ -1,10 +1,10 @@
 @include('kontour::buttons.hamburger')
-<ul data-kontour-widget="menu">
+<ul data-kontour-widget="menu" role="list">
 @foreach($links as $heading => $headingLinks)
   @if(count($headingLinks))
     <li>
       <small>{{ $heading }}</small>
-      <ul>
+      <ul role="list">
       @foreach($headingLinks as $link)
         <li{!! preg_match('#'.$link->getUrl().'#', url()->full()) ? ' aria-current="true"' : '' !!}>{{ $link }}</li>
       @endforeach
diff --git a/resources/views/widgets/message.blade.php b/resources/views/widgets/message.blade.php
index a38ec65d..ee43edbc 100644
--- a/resources/views/widgets/message.blade.php
+++ b/resources/views/widgets/message.blade.php
@@ -1,5 +1,5 @@
 <section data-kontour-widget="message">
-  <ol>
+  <ol role="list">
     @foreach($messages as $message)
       <li
         data-kontour-message-level="{{ $message['level'] }}"
diff --git a/resources/views/widgets/personalRecentVisits.blade.php b/resources/views/widgets/personalRecentVisits.blade.php
index faf4a4fe..1b55adbe 100644
--- a/resources/views/widgets/personalRecentVisits.blade.php
+++ b/resources/views/widgets/personalRecentVisits.blade.php
@@ -1,7 +1,7 @@
 @if(count($visits))
   <aside data-kontour-widget="personalRecentVisits">
     <header>{{ trans('kontour::widgets.personalRecentVisits.title') }}</header>
-    <ul>
+    <ul role="list">
     @foreach($visits as $visit)
       @if(url()->full() != $visit->getLink()->getUrl())
         <li data-kontour-visit-type="{{ $visit->getType() }}"{!! empty($visit->getLink()->getDescription()) ? ' title="' . e($visit->getLink()->getName()) . '"' : '' !!}>{{ $visit->getLink() }}</li>
diff --git a/resources/views/widgets/teamRecentVisits.blade.php b/resources/views/widgets/teamRecentVisits.blade.php
index e2936028..f0e49e2d 100644
--- a/resources/views/widgets/teamRecentVisits.blade.php
+++ b/resources/views/widgets/teamRecentVisits.blade.php
@@ -1,7 +1,7 @@
 @if(count($visits))
   <aside data-kontour-widget="teamRecentVisits">
     <header>{{ trans('kontour::widgets.teamRecentVisits.title') }}</header>
-    <ul>
+    <ul role="list">
     @foreach($visits as $visit)
       <li data-kontour-visit-type="{{ $visit->getType() }}" data-kontour-username="{{ $visit->getUser()->getDisplayName() }}"{!! empty($visit->getLink()->getDescription()) ? ' title="' . e($visit->getLink()->getName()) . '"' : '' !!}>{{ $visit->getLink() }}</li>
     @endforeach
diff --git a/tests/Feature/UserlandControllerTest.php b/tests/Feature/UserlandControllerTest.php
index fc218242..54aef82e 100644
--- a/tests/Feature/UserlandControllerTest.php
+++ b/tests/Feature/UserlandControllerTest.php
@@ -53,7 +53,7 @@ public function test_menu_widget()
         $response = $this->actingAs($this->user, 'admin')->get(route('userland.index'));
 
         $response->assertSuccessful();
-        $response->assertSee('<ul data-kontour-widget="menu">', false);
+        $response->assertSee('<ul data-kontour-widget="menu" role="list">', false);
         $response->assertSee('>main<', false);
         $response->assertSee('<a href="' . route('userland.index') . '" aria-current="page">Userland Tool</a>', false);
         $response->assertSee('>External<', false);