♿ feat: implement accessibility improvements#446
♿ feat: implement accessibility improvements#446ana-maria-silva wants to merge 1 commit intoglobocom:masterfrom
Conversation
| </h1> | ||
| <div className="hero__description"> | ||
| Megadraft is a Rich Text editor built on top of | ||
| <header>Megadraft is a Rich Text editor built on top of |
There was a problem hiding this comment.
Não acho que aqui seja a melhor opção pra adicionar o header, poderia rever o uso pra uma div superior? Se atentar a indentação também nesse caso
There was a problem hiding this comment.
Necessário revisar o código do projeto, para implementação de melhorias para acessibilidade e usabilidade de acordo com as diretrizes da web semântica.
There was a problem hiding this comment.
Realmente, revisando esta sugestão, neste caso, o mais recomendado seria <h2> seguindo a hierarquia de títulos, assim como para toda a estrutura utilizando <main>, <article> ,<aside>, <figure> e <nav> para o menu.
| @@ -1,5 +1,5 @@ | |||
| <!DOCTYPE html> | |||
| <html> | |||
| <html lang="pt-br"> | |||
There was a problem hiding this comment.
O default idioma aqui esta pt-br mas todo o conteúdo é em inglês
| </h1> | ||
| <div className="hero__description"> | ||
| Megadraft is a Rich Text editor built on top of | ||
| <header>Megadraft is a Rich Text editor built on top of |
There was a problem hiding this comment.
Muito bem observado, mas está quase lá!
Considerando boas práticas de acessibilidade,
o texto não deveria estar dentro de uma tag apropriada, como h, p, span, etc.
O que acha?
There was a problem hiding this comment.
Correto, neste caso, o mais recomendado seria <h2> seguindo a hierarquia de títulos, assim como para toda a estrutura utilizando <main>, <article> e <aside> para o conteúdo, <figure> para a imagem e <nav> para o menu.
ana-maria-silva
changed the title
ana-maria-silva
changed the title
ana-maria-silva
changed the title
4 participants
Related Issue
Proposed Changes
Melhorias de acordo com diretrizes de acessibilidade web:
<header> <main <section>