Muito satisfeito com meu desempenho e muito grato pela oportunidade de esta estudando com vocês
João Xavier
@joaoxavier-profissionalAll comments
- @marcelogitfrancaWhat specific areas of your project would you like help with?@joaoxavier-profissional
Parabéns pela solução, o css ficou ótimo! Não se esqueça de fazer o estilo mobile, caso ainda não saiba, aqui no frontend mentor logo logo ira aprender a usar media queries que ira possibilitar a fazer isso.
Marked as helpful - @TerezLWhat specific areas of your project would you like help with?
Any tips how to simplify css and js would be appreciated
@joaoxavier-profissionalThe script is working propperly expect by using negative numbers, try using grid layout for css, it will make your life pretty easier with this project.
- @SamaraOliveira07@joaoxavier-profissional
Excelente ideia de usar o produto Jequiti! Notei que você usa bastante o seletor '''>''' para falar com o filho de alguma classe ou div, olhando pelo css, da para perceber que não se faz necessário. Uma outra dica é de usar medidas relativas como rem para fontes e não defina tamanhos para containers a não ser se for realmente necessário, recomendo bastante esse vídeo aqui https://www.youtube.com/watch?v=x4u1yp3Msao&t=60s
Marked as helpful - @MateuszZalew@joaoxavier-profissional
Very good!
- @canbld@joaoxavier-profissional
Awesome!
- @AmirxonEngineer@joaoxavier-profissional
Seeing your code i noticed that it uses column-gap, maybe using just gap: 5% would make your layout a bit more consistent.
Marked as helpful - @iamzahidhasan239@joaoxavier-profissional
very good!
- @TofeDevWhat are you most proud of, and what would you do differently next time?
I'm proud of how I was able to display the images in both desktop and mobile
What challenges did you encounter, and how did you overcome them?There were some CSS property I didn't remember so I needed to look a lot in the mdn web docs until I got everything I needed
What specific areas of your project would you like help with?Any help is welcomed!
@joaoxavier-profissionalAwesome page!
- @SteveNoyes@joaoxavier-profissional
pretty good!
- @NaoMauss@joaoxavier-profissional
Very good
- @AlvaroPratesWhat are you most proud of, and what would you do differently next time?
I'm proud that I added comments to organize the CSS a little better (compared to the last qr-code project). I've also tried to use the CSS multiselector to compress the styling code.
What challenges did you encounter, and how did you overcome them?I'm a little more confortable with basic flexbox, but I believe that with the next projects this should improve more.
@joaoxavier-profissionalpretty good!
- @eliasgiannantonio@joaoxavier-profissional
Set the description div as display: none, it will make the div and the content inside it disappear, pretty good tho
Marked as helpful