@ralphvirtucio
Posted
Hi Jaiden Lee, Great job on completing this challenge!
Here are my feedbacks to your challenge:
-
It's not a good practice to have two
<h1></h1>
tag element in your HTML, I'd suggest making the other heading a<h2></h2>
-
If your struggling on positioning the image, I'd suggest wrap the image in the div element, set a align-self property with a value of flex-end to the div selector.
-
Try using rem or em for your max-width and media-query. This will improve responsiveness of the page because rem and em are relative units they scale with the user's font size settings.
-
I'd suggest to remove the property text-align in your .site-container and move it to each content that you want to have the text centered. Setting a text-align: center in the parent div will cause everything to be centered. That's the reason that the text in your card are also centered.
** HAPPY CODING 🧑💻🧑💻 CODE UP ☕☕☕**