Abdallah Noor• 150
@AbdallahNoor
Submitted
please share your thoughts on what i should improve ;)
César Palma
@xsrpm
All comments
- César Palma• 425
@xsrpm
Posted
- You could adapt your website for mobile devices.
- I'm curious, with what tool did you model your website?
Marked as helpful
1 @JSaporski
Submitted
Any suggestion would help me a lot. Feel free to leave a comment :D
César Palma• 425@xsrpm
Posted
the font color of the header button appears to be white instead of hsl (204, 43%, 93%), otherwise it is almost identical. You can separate your css into files by target.
Marked as helpful
1- Gian Franco• 290
@gianback
Submitted
Hello everyone, it is my first time uploading a project, I hope to learn a lot from your comments!
César Palma• 425@xsrpm
Posted
You can separate the css code by target, it is a good practice.
<link rel = "stylesheet" href = "style.css" /> <link rel = "stylesheet" href = "desktop.css" media = "screen and (min-width: 1440px)" /> You can check the uploads of others to review where to improve.Marked as helpful
1 - VIPenn• 10
@pennvi
Submitted
If you have a moment, please offer comments! I'd really appreciate it! Also, I've since realized that I should not structure my CSS entirely with IDs (lol), and I will not do it henceforth! And also, I know the responsiveness is a bit shotty/sloppy, but, quite honestly, I just wanted to submit this and begin working on another one, as I spent more time on this than I'd like to admit. Thank you!
César Palma• 425@xsrpm
Posted
The design is almost identical. Regarding what you comment on the use of ids, it is true you should not use them in this type of exercise, just to obtain a reference to a DOM element in javascript. In css with classes and specificity with tags it is enough. The BEM methodology also helps to correctly name the classes.
1 - Meenakshi K• 570
@Memeena
Submitted
Please review my solution and provide your valuable feedback!
César Palma• 425@xsrpm
Posted
Don't forget to use semantic tags like main and footer. Your solution is visually very similar
0 - Marialba Córdova• 60
@Marialba16
Submitted
Hi guys, how are you? This is my third challenge, as in the previous ones, I would like to read your comments about how I can improve or what I should avoid when coding. Thank you!!
César Palma• 425@xsrpm
Posted
hello some feedbacks: 1- instead of
<div class = "container"> uses: <main> 2- instead of: <div class = "attribution"> uses <footer class = "attribution">Marked as helpful
0 - septem• 30
@myatthu000
Submitted
any feedback is appreciated ......////
César Palma• 425@xsrpm
Posted
be careful with the hover on the button, it is increasing the size of the container
0
