search

dwyl / mvp

📲 simplest version of the @dwyl app
https://mvp.fly.dev
GNU General Public License v2.0
87 stars 2 forks source link

PR: Improve Accessibility #129 #130

Closed nelsonic closed 2 years ago

nelsonic commented 2 years ago

I've tried to follow the a11y guidelines outlined in #129

image

codecov[bot] commented 2 years ago

Codecov Report

Merging #130 (36c3857) into main (7f1766c) will not change coverage. The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #130   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            7         7           
  Lines          137       137           
=========================================
  Hits           137       137
Impacted Files Coverage Δ
lib/app_web/live/app_live.ex 100.00% <ø> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

nelsonic commented 2 years ago

Thanks for checking. Second <main> tag removed from lib/app_web/templates/layout/live.html.heex ✅

nelsonic commented 2 years ago

Merci @SimonLab 🙌