Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Simplificando marcação <main> #419

Closed
wants to merge 1 commit into from
Closed

Conversation

hansmosl
Copy link
Contributor

@hansmosl hansmosl commented Sep 7, 2016

No description provided.

@claudiosanches
Copy link
Member

Isso apenas dessa forma quebra a navegação de acessibilidade.

@hansmosl
Copy link
Contributor Author

hansmosl commented Sep 11, 2016

Desculpe ter de insistir nesse ponto @claudiosmweb, mas à luz da explicação abaixo peço que reconsidere minha sugestão.

Ao analisar o código do single.php que consta neste momento no Projeto é possível perceber que o ID de navegação consta como "main-content" sendo que o link de acesso rápido em header.php (linha 24) aponta para "content". Dessa forma o atalho de acessibilidade não funcionará.

Além disso o uso da div com id "primary" só ocorre no single.php e no video.php. Em todos os demais arquivos de modelo abaixo descritos a marcação simplificada do elemento html "main" já consta em uso.

À saber:

index.php
page.php
page-sidebar.php
category.php
tag.php
image.php
404.php
search.php

hansmosl added a commit to hansmosl/odin that referenced this pull request Sep 29, 2016
Volto a propor essa mudança nesse arquivo tomando como base a explicação abaixo.

O single.php como está apresenta o ID de navegação "main-content" sendo que o link de acesso rápido em header.php (linha 24) aponta para "content". Dessa forma o atalho de acessibilidade não funcionará.

O envelopamento do main dentro da div "primary" só ocorre no single.php e no video.php. Também já propus uma correção  para isso na <a href="https://github.com/wpbrasil/odin/pull/419">pull wpbrasil#419 </a>. Ao meu ver a mudança é válida. Não sendo gostaria de saber apenas o motivo. pois em todos os demais arquivos de modelo abaixo descritos a div "primary" já foi removida.

index.php
page.php
page-sidebar.php
category.php
tag.php
image.php
404.php
search.php
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants