Emal
@syeero7All comments
- @dhiaa-zerSubmitted 2 months ago@syeero7Posted 2 months ago
Well done. Consider using a variable font instead of multiple font files.
0 - @AdebukolSubmitted 3 months ago@syeero7Posted 3 months ago
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 helpful0 - @mohammedsabSubmitted 3 months ago@syeero7Posted 3 months ago
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 - @JEWebDevSubmitted 4 months agoWhat 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!
@syeero7Posted 4 months agoWell done!
You can use
<time>
for the publish date inside<p>
. also, please note that the-webkit-font-smoothing
property only works on macOS. The page did not render well on my browser.Marked as helpful0 - @kevinphucSubmitted 4 months ago@syeero7Posted 4 months ago
well done! you can use fix some accessibility errors by using a <main> element instead of <div> & one <h1> instead of <p>
Marked as helpful0