@MohamedAridah
Posted
Hello @AbdulkerimAwad, Yo did good job for this challenge🎉👍. Your website is Responsive and that's really cool.
However i have some notes for you:
-
Don't use compiled css files for learning purpose projects. it will be impossible to read your code form
style.css
file. -
css nesting rules are so long. and this will make browser take time to render it. example for what i mean:
html body .main-container header .next-generation button
it could be shortened to
.main-container header .next-generation button
-
try use
transition
property specially for.next-generation button
and footer links. -
change
.card-title
color
property of the articles on hover
I hope this wasn't too long for you, hoping also it was useful😍.
Goodbye and have a nice day.
Keep coding🚀
Marked as helpful
@AbdulkerimAwad
Posted
@MohamedAridah I have made the appropriate modifications. You can see them if you like. Thank you very much for your opinions and suggestions
@MohamedAridah
Posted
@AbdulkerimAwad Good job. if you like you can see My solution for this challenge it may be useful for you. I'm also advising you to start with simple challenges I myself did that it will be very useful for you when starting work with big challenges.
wish you good luck with the next project👍🌹
Marked as helpful