@mojtaba-mehrzadSubmitted 1 day ago
Leonardo Almeida
@leonardoalmeida7All comments
- @leonardoalmeida7Posted about 15 hours ago
Great, the code is very easy to understand and can use some more complex concepts in js and css well.
0 - @ivandoliveiraSubmitted 7 days agoWhat are you most proud of, and what would you do differently next time?
Estou orgulhoso de ter conseguido usar as variáveis de cores.
@leonardoalmeida7Posted 7 days agoGostei da forma como você usa variáveis, fica uma dica pra você, use mais tags semântica na estrutura do HTML assim ganhara mais visibilidade nos mecanismos de busca e também ficara bem estruturado o código, tags como footer e section são algumas das tags que poderia ter usado no código.
0 - @FriedHammSubmitted 7 days ago@leonardoalmeida7Posted 7 days ago
the code looks good to me, but it lacks mobile design and a little more interaction with javascript
0 - @SurajCaseySubmitted 12 days agoWhat are you most proud of, and what would you do differently next time?
This is the first project that I have used Javascript properly. I had to do lot of research to make button respond. I explored a lot after this project. I will try to make seperate css file for different container to make code readable..
What challenges did you encounter, and how did you overcome them?share-wrapper was using space after it was hidden which i could not figure out how to fix but i did research and fixed it. I learned different css shapes as I had to make triangle at the bottom.
What specific areas of your project would you like help with?I want help to write a clean code. Also, I want recommendation if I have to start writing code starting with moble screen sizes first.
@leonardoalmeida7Posted 12 days agoThe HTML code is very clean but lacks semantics, the CSS structure is very well written, and the JavaScript code looks good to me.
1 - @Maame-codesSubmitted 14 days ago@leonardoalmeida7Posted 14 days ago
There was a lack of semantics in the HTML code, the CSS is well commented and looks clean, I tested it on a small screen but it wasn't very good, a tip is to better adjust the code structure using a media query to solve it.
0 - @ttwmfSubmitted 16 days ago@leonardoalmeida7Posted 16 days ago
the code is semantic and clean, but the elements don't seem well positioned, details like width, font and alignment would need a little more attention.
0 - @SydsBikeSubmitted 17 days agoWhat challenges did you encounter, and how did you overcome them?
It was challenging to create the mobile version. I ran into some issues with proper element placement but eventually overcame these challenges by utilizing the MDN and learning more about the nuances of CSS.
What specific areas of your project would you like help with?All feedback welcome
@leonardoalmeida7Posted 17 days agothe code is semantic and not very confusing, I tested it on some screens, on the desktop version it looks good but on mobile it needed to adjust the element better, reducing it by 25%, so there would be space left on the sides of the screen.
Marked as helpful0 - @kareemsakrSubmitted 18 days ago@leonardoalmeida7Posted 18 days ago
the page was responsive and met the semantics well
Marked as helpful0
