dhiaa-zer• 60
@dhiaa-zer
Submitted
Emal
@syeero7
All comments
- Emal• 110
@syeero7
Posted
Well done. Consider using a variable font instead of multiple font files.
0 @Adebukol
Submitted
Emal• 110@syeero7
Posted
Consider using CSS variables, as they can enhance maintainability in styling. Additionally, use semantic markup, as it is important for HTML elements to indicate what they actually are, rather than how they appear in the browser
Marked as helpful
0- mohammed saber• 50
@mohammedsab
Submitted
Emal• 110@syeero7
Posted
Well done.
<li>element is not keyboard-focusable. Use thea(anchor) element inside the<li>tag.for example:
<li><a href="#"> link </a></li>.0 - Juan• 480
@JEWebDev
Submitted
What are you most proud of, and what would you do differently next time?
I'm proud of finishing this project in less than 1 hour and 30 min. Also i learned something new with the css clamp method.
What challenges did you encounter, and how did you overcome them?
Adjusting the font sizes without media queries, it was challenging and fun!
What specific areas of your project would you like help with?
Any feedback is welcome. Have a great day!
Emal• 110@syeero7
Posted
Well done!
You can use
<time>for the publish date inside<p>. also, please note that the-webkit-font-smoothingproperty only works on macOS. The page did not render well on my browser.Marked as helpful
0 - Vĩnh Bảo Phúc• 30
@kevinphuc
Submitted
Emal• 110@syeero7
Posted
well done! you can use fix some accessibility errors by using a <main> element instead of <div> & one <h1> instead of <p>
Marked as helpful
0
