Submitted over 4 years ago
fylo-dark-theme-landing-page using HTML and CSS
@varunbhalla19
Design comparison
SolutionDesign
Solution retrospective
Please review and provide your valuable feedback.
Community feedback
- @rfilenkoPosted over 4 years ago
Good job, !importart is considered a bad practice, so just use classes for styling - easy to maintain and simple overwrite. Also add hover states, links in footer (including tel and email).
Regards, Roman
2@abdulsalamcodesPosted over 4 years agoI strongly agree with this.Using classes also allow for flexibility and make debugging easier.
1@varunbhalla19Posted over 4 years agoI have now added hover state styles in footer, and removed !important usage in styling. Thanks a lot :)
0@rfilenkoPosted over 4 years ago@varunbhalla19 hover state should be only on links (<a> tag) or buttons :)
0
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