fix(header): improve mobile menu accessibility #157
Conversation
|
Deploy preview for portal-ascepa ready! ✅ Preview Built with commit db65d3d. |
…chexpertspro/portal-ascepa into fix/mobile-menu-accessibility
There was a problem hiding this comment.
Fala @patzsl! Testei o menu pelo Deploy preview e por algum motivo o botão do menu não está aparecendo no mobile
|
There was a problem hiding this comment.
Agora ficou bem massa! Só um ponto que poderia ser aprimorado é deixar o texto "Menu" clickável, como se ele fizesse parte do botão. Um exemplo legal para se basear é o menu mobile da W3C:
https://www.w3.org/
| fill="none" | ||
| xmlns="http://www.w3.org/2000/svg" | ||
| <div id="menu-toggle-container"> | ||
| <h2 id="menu-title" class="mr-1rem">Menu Principal</h2> |
There was a problem hiding this comment.
- Caso seja necessário usar uma tag, recomendaria usar o span, pois o h2 deixou a fonte muito grande.
- Uma outra sugestão seria colocar o texto, com ou sem tag(conforme necessidade), dentro da tag button, pois assim não precisaria colocar a tag div envolvendo todo mundo. Quanto menos tag melhor : D
|
gente, reparei quando usei dark mode para mudar fundo preto, mas menu ainda não foi alterado para fundo preto, ou seja, está com fundo branco, então, gostaria de saber que vai resolver aqui ou criar novo issue? segue print |
Excelente ponto @Flamilani! Acho que seria interessante fazer a correção nessa PR mesmo. |
Descrição
Adiciona titulo e acessibilidade no menu mobile, como descrito na task de a11y #156
Tipo da alteração
fix: Correção de acessibilidade
Checklist