Merge pull request #21 from ndarilek/misc-a11y

Add additional region for search region when mobile sidebar is displayed.
This commit is contained in:
Chris McCord 2022-01-05 16:02:08 -05:00 committed by GitHub
commit dff06cf171
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -1,4 +1,4 @@
<div id="mobile-sidebar-container" class="fixed inset-0 flex z-40 lg:hidden" aria-modal="true" style="display: none;">
<div id="mobile-sidebar-container" class="fixed inset-0 flex z-40 lg:hidden" aria-modal="true" style="display: none;" role="region">
<div
class="fixed inset-0 bg-gray-600 bg-opacity-75"
phx-click={hide_mobile_sidebar()}>
@ -49,7 +49,7 @@
<!-- Static sidebar for desktop -->
<div class="hidden lg:flex lg:flex-shrink-0">
<div class="hidden lg:flex lg:flex-shrink-0" role="region">
<div class="flex flex-col w-64 border-r border-gray-200 pt-5 pb-4 bg-gray-100">
<div class="flex items-center flex-shrink-0 px-6">
<.link navigate={home_path(@current_user)}>
@ -95,7 +95,7 @@
<!-- Main column -->
<div class="flex flex-col w-0 flex-1 overflow-hidden">
<!-- Search header -->
<div class="relative z-10 flex-shrink-0 flex h-16 bg-white border-b border-gray-200 lg:hidden">
<div class="relative z-10 flex-shrink-0 flex h-16 bg-white border-b border-gray-200 lg:hidden" role="navigation">
<button type="button"
id="show-mobile-sidebar"
aria-expanded="false"