@romila2003
Posted
Hi Olalekan,
Congratulations 🎉 for completing this challenge, it was a great attempt.
Some issues I noticed within your design and code include:
- It is best practice to wrap the main content within the
main
tag e.g.<main class="container"></main>
. - You are missing the
font-family
property within your texts which is provided within the style guide. - The button is missing the
cursor
property e.g.cursor: pointer;
- The color of the button is different however when you hover over the button, it shows the same color as your current button color.
- Your button is missing the
type
attribute e.g.<button type="submit" class="btn"></button>
- Regarding the responsiveness of the card, it looks good however you do not need many media queries, especially within this project therefore It would be fine to have 1 or 2 media queries to change the layout when the card shifts into the desktop screen.
Overall, great attempt and wish you the best of luck for your future projects so keep coding 👍.
Marked as helpful
@OlaOluwalekan
Posted
@romila2003 Thanks for the feedback. I really appreciate. I'll heed to your recommendations