Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Huddle landing page with single introductory section

Elif Seyhanโ€ข 180

@elifseyhan


Design comparison


SolutionDesign

Community feedback

@catherineisonline

Posted

HI! Your solution looks nice though there are a couple of things you can improve which I hope will be helpful! ๐Ÿ˜Š

In the social media div, you need to place the icons inside a tag, because the purpose of a tag is to redirection to other social media platforms, in this case, it's not just decoration.

<div class="footer-icons">
			<i class="fa-brands fa-facebook-f"></i>
			<i class="fa-brands fa-twitter"></i>
			<i class="fa-brands fa-instagram"></i>
		</div>

I would also add some transitions for active states (when colors change on hover). It creates more interactivity and makes the project look cooler. Active states can be done on buttons, links, titles which act like links, or anything else, you choose.

You can read more about it here, in case you havenโ€™t done much of it:โ€จhttps://www.w3schools.com/css/css3_transitions.asp

Also, a quick question, why did you need class for h1, for example? Because you have only one h1 on the page so you don't need any class to target this tag for CSS, for example.

Marked as helpful

1

Elif Seyhanโ€ข 180

@elifseyhan

Posted

@catherineisonline Hi! Thank you for useful suggestions. :) I made some changes on my codes. I think it looks better thanks to you.

Happy coding ๐Ÿฅฐ

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord