Boris• 2,870
@mkboris
Posted
Nice work @Jorge-234, here are some suggestions
- Consider using the HTML Table element for the nutrition table instead of divs. This approach offers a better semantic structure.
- All content should be wrapped within landmarks. Wrap a
main
tag around the .container and afooter
for the attribution. - It's best practice linking Google fonts directly in the HTML
head
section rather than directly in your CSS file as it enables asynchronous downloading, improving page load times. - Consider using a modern CSS reset at the start of the styles in every project. Like this one Modern CSS Reset. This will help reset a list of default browser styles.
- Font-size should be written in
rem
not px. This article explains it better Why font-size must NEVER be in pixels.
Hope this helps, Good luck!
1
Jorge-234• 110
@Jorge-234
Posted
@mkboris Thank you very much for your insightful feedback. I truly appreciate your suggestions and am excited about the opportunity to apply them into my next project.
0