Wendy• 1,890
@wendyhamel
Submitted
Alberdam Santiago Sancler da Silva
@hunter-code
All comments
@hunter-code
Posted
Oi, Wendy muuuito boa a sua solução pois , vejo que você foi muitíssimo próximo da solução colocada como desafio. 1 - A sua solução teve bastante regra semântica e ficou bem aplicadas com as tags responsáveis; 2 - A sua solução tem a presença de código com bastante acessibilidade e ficou muito bom ; 3 - O layout , fica vem situado em diferentes dispositivos ; 4 - E também pelo que vi o código é bem estruturado , legível e tem reaproveitamento; 5 - A solução não tem diferênça aparente do projeto, na minha opinião perfeito!!! Continue evoluindo.
0- Felipe Louzeiro• 10
@felipelouzeiro
Submitted
What are you most proud of, and what would you do differently next time?
Gostei de aplicar técnicas de acessibilidade no código, na próximo poderia usar scss, por exemplo.
What challenges did you encounter, and how did you overcome them?
Trabalhar com diferentes tamanhos de imagem foi algo novo.
@hunter-code
Posted
Oi, Felipe sua solução ficou muito, sensacional, bem próxima da original. Observei um HTML bem semântico e também as tags bem precisas. Notei também a preocupação com a acessibilidade, gostei de ver. Observei a presença do atributo alt, indicando o texto alternativo. Eu notei Felipe que quando eu abri sua solução no note-book , sansung - DESKTOP-U1IJMO7, a imagem do perfume não atingiu a altura total da divisão que a pertence, e o botão está um pouco grande e seu texto parece estar com um tamanho exedente, neste dispositivo. Mas a solução está bem robusta ao que propõe o projeto. O código está bem reutilizável e estruturado também, muito bom!!! A slução está bem próxima do design ! Obs : É interessante também , quando se trata de design, seguir o máximo de igualdade do produto. Tente colocar a marcação da modeda que foi passado , ex: $, ao invés de U$$. É interessante eu citar que isso só aparece para mim no site configurado em português.
0 - ICode88• 280
@ICode88
Submitted
@hunter-code
Posted
Muito boa solução Código I88, pelo que vi, muito semântico , e também boa responsividade, atente somente, para alguns dispositivos maiores, para que possam ter um bom espaçamento. Seu código está bastante legível, e reutilizável. Continue evoluindo sempre meu companheiro.
0 - Keethesh• 50
@keethesh
Submitted
@hunter-code
Posted
Oi, Keethesh, tudo bem ? Vi a sua solução e está muito bacana, muitísso próxima da original, gostei de mais ! Você também pode tentar a aplicação da tag main, no projeto que deixa o seu código html mais semântico. Tente tamém observar a sua aplicação em outros tamnhos de tela , para que sua responsividade melhore a cada dia, para saber mais sobre respondividade leia neste site, https://developer.mozilla.org/pt-BR/docs/Web/CSS/CSS_media_queries/Using_media_queries. Continue evoluindo sempre meu irmão.
0 What are you most proud of, and what would you do differently next time?
/
What challenges did you encounter, and how did you overcome them?
I had some problems with the category that was being full width. but that is fixed with align-self.
What specific areas of your project would you like help with?
Just more help in sizing, and best practices.
@hunter-code
Posted
Oi, gajbos99 , tudo bem ? Sua solução ficou o máximo . Talvez também seria interessante englobar o conteúdo principal sobre a tag main, e em seguida aplica-lo. utilizando a regra de posicionamento nativo do css é bem interessante para obtermos o layout em vários tamanhos de telas de uma forma simples em aplicações pequenas. A solução ficou muito bem próxima do design.
0- Mateogr03• 310
@Mateogr03
Submitted
@hunter-code
Posted
Muito boa solução Mateogr03, todos os requisitos à cima são correspondidos. Continue progredindo e dando o máximo em seus designes.
0
