FernandaBernardo
commented
6 years ago Bom dia @danilogila 😊
Vou tentar montar meu feedback aqui, qualquer dúvida estou à disposição e aberta também para ouvir, são apenas sugestões :)
- não existe um
<h1>na sua página, e acredito que essa informação semanticamente é bem importante. Nesse caso, acho que poderia ser o logo - Semanticamente falando, você aqui que o tipo do filme e a "|" seria um título? https://github.com/danilogila/MovieRise/blob/master/index.html#L40
- Essa
<div>é necessária? https://github.com/danilogila/MovieRise/blob/master/index.html#L54 - Porque essas listas são uma
<ol>? Ela é ordenada? A ordenação é obrigatória? https://github.com/danilogila/MovieRise/blob/master/index.html#L56, https://github.com/danilogila/MovieRise/blob/master/index.html#L1029, https://github.com/danilogila/MovieRise/blob/master/index.html#L1044 - Esses inputs/labels não poderiam ser
<button>? https://github.com/danilogila/MovieRise/blob/master/index.html#L135 - Se essa
<div>não tem conteúdo, ela não deveria ser colocada pelo JS? https://github.com/danilogila/MovieRise/blob/master/index.html#L151 - Precisa dessa
<div>? A classecopyrightnão poderia estar na<nav>de dentro? https://github.com/danilogila/MovieRise/blob/master/index.html#L1042 - No mobile não aparecem os outros menus, só o que está selecionado
- Cuidado ao colocar
<br>porque estará SEMPRE quebrado.
São algumas coisas, alguns detalhes, mas seu código está bom, parabéns!
danilogila
Gostaria de opiniões a respeito do que posso melhorar no meu código e como deixar ele mais semântico possível.
Repositório: https://github.com/danilogila/MovieRise Link: http://teste-movierise.surge.sh/
Obrigado a quem puder ajudar rs