@marcelogitfrancaSubmitted 3 months ago
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-profissionalAll comments
- @joaoxavier-profissionalPosted 3 months ago
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 helpful0 - @TerezLSubmitted 3 months agoWhat specific areas of your project would you like help with?
Any tips how to simplify css and js would be appreciated
@joaoxavier-profissionalPosted 3 months agoThe 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 - @SamaraOliveira07Submitted 3 months ago@joaoxavier-profissionalPosted 3 months ago
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 helpful0 - @MateuszZalewSubmitted 3 months ago
- @canbldSubmitted 3 months ago
- @AmirxonEngineerSubmitted 4 months ago@joaoxavier-profissionalPosted 3 months ago
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 helpful2 - @iamzahidhasan239Submitted 5 months ago
- @TofeDevSubmitted 3 months agoWhat 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!
- @SteveNoyesSubmitted 7 months ago
- @NaoMaussSubmitted 3 months ago
- @AlvaroPratesSubmitted 3 months agoWhat 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.
- @eliasgiannantonioSubmitted 3 months ago@joaoxavier-profissionalPosted 3 months ago
Set the description div as display: none, it will make the div and the content inside it disappear, pretty good tho
Marked as helpful1
