Skip to content
OverviewSolutions
5
Comments
5
Emal
@syeero7

All comments

  • Emal•110
    @syeero7Posted 3 months ago

    Well done. Consider using a variable font instead of multiple font files.

    Variable fonts

    0
  • Emal•110
    @syeero7Posted 4 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 helpful
    0
  • Emal•110
    @syeero7Posted 4 months ago

    Well done. <li> element is not keyboard-focusable. Use the a (anchor) element inside the <li> tag.

    for example: <li><a href="#"> link </a></li>.

    0
  • Juan•480
    @JEWebDevSubmitted 4 months ago
    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!

    Responsive blog preview card with vanilla HTML and CSS

    2
    Emal•110
    @syeero7Posted 4 months ago

    Well 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 helpful
    0
  • Emal•110
    @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 helpful
    0