@mkboris
Posted
Hi Divadovitch, great job on completing this challenge, overall this looks great.
The hover is applied only to the title and not the card as a whole and for the title being a link I think it's suitable to place an anchor tag around it.
Some suggestions.
- 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.
- 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.
Hope this helps.
Marked as helpful
@Divadovitch
Posted
@mkboris
Hi Boris,
Thank you for these suggestions, I have added the link to my favorites and I have taken note of the loading of Google fonts in the HTML head
.