Submitted over 2 years ago
Huddle landing Page Simple HTML and Css
@NaveenGumaste
Design comparison
SolutionDesign
Solution retrospective
I took 2 months break, Now I'm little rusty so i am "Open for Feed back"
Community feedback
- @jasurkurbanovPosted over 2 years ago
Firstly, make naming understandable for other developers, who will join to the project.
You created class imgs, which is confusing. Since I expect from code it should be either
<header>
tag or class name of header.Secondly, inside class "social" you're using too much <div> tag. Instead you can simply
<div class="social"> <a href="#" target="_blank"> <ion-icon name="logo-facebook"></ion-icon> </a> <a href="https://twitter.com/crazi_monk" target="_blank"> <ion-icon name="logo-twitter"></ion-icon> </a> <a href="https://github.com/Crazimonk" target="_blank"> <ion-icon name="logo-instagram"></ion-icon> </a> </div>
Here you can adjust your social icons on left using Flexbox
Marked as helpful0
Please log in to post a comment
Log in with GitHubJoin 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