marcelogitfranca• 40
@marcelogitfranca
Submitted
What specific areas of your project would you like help with?
Muito satisfeito com meu desempenho e muito grato pela oportunidade de esta estudando com vocês
João Xavier
@joaoxavier-profissional
All comments
- João Xavier• 230
@joaoxavier-profissional
Posted
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
0 - Terez Lagova• 380
@TerezL
Submitted
What specific areas of your project would you like help with?
Any tips how to simplify css and js would be appreciated
João Xavier• 230@joaoxavier-profissional
Posted
The 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.
0 - Aleksei Bodeev• 370
@alekseibodeev
Submitted
What challenges did you encounter, and how did you overcome them?
I did introduce an extra challenge for myself. I implemented fake network delay on data fetching to add it a "real" feel. To avoid empty screen and layout shifts on data fetching I built a skeleton screen.
João Xavier• 230@joaoxavier-profissional
Posted
great project, but function fakeNetwork just made it slower.
0 - Samara Lobato• 50
@SamaraOliveira07
Submitted
João Xavier• 230@joaoxavier-profissional
Posted
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
0 - João Xavier• 230
@joaoxavier-profissional
Posted
Very good!
1 - Emir Can BULDU• 110
@canbld
Submitted
João Xavier• 230@joaoxavier-profissional
Posted
Awesome!
0 - AmirkhonKhaitov• 80
@AmirxonEngineer
Submitted
João Xavier• 230@joaoxavier-profissional
Posted
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
2 - Zahid Hasan• 150
@iamzahidhasan239
Submitted
João Xavier• 230@joaoxavier-profissional
Posted
very good!
0 - Luca• 120
@TofeDev
Submitted
What 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!
João Xavier• 230@joaoxavier-profissional
Posted
Awesome page!
0 - Steven Noyes• 270
@SteveNoyes
Submitted
João Xavier• 230@joaoxavier-profissional
Posted
pretty good!
0 - NaoMauss• 30
@NaoMauss
Submitted
João Xavier• 230@joaoxavier-profissional
Posted
Very good
0 - PAlvaroPrates• 110
@AlvaroPrates
Submitted
What 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.
João Xavier• 230@joaoxavier-profissional
Posted
pretty good!
0
