@Ishmaelsealey
Posted
Hey Tatiana!
Great job on completing the challenge!
Below I have suggestions that I think you will find helpful.
- For accessibility purposes, your webpage could contain one
<h1>
tag. - The button does nothing when I hover over it, however it changes color when I click on it. use the pseudo class
:hover
on an object to add styles when the cursor hovers over it. More information about pseudo classes here! - The attribution class does not need margin to center the text. Use the
text-align: center;
style to do this. - Your html uses many
<div>
elements and while it does get the job done, some of them are not necessary. In the future I suggest you take time before beginning to style your webpage, to put everything in a suitable html element to form a base to build your webpage on. - Your website is not mobile responsive. Using
@media screen (max-width: 1000px) { ...mobile styles here... }
will help you do this. For instance, you used display flex on the card class, to make a mobile version, you can use theflex-direction: column;
style to change the layout. W3 schools has a lovely article that discusses media queries. Check them out here!
I do hope you find my suggestions helpful! Happy coding!
Marked as helpful