Design comparison
Solution retrospective
This definitely felt a fair bit tougher than the previous ones I did. On top of that, I took a break in the middle of this project and the code got out of hand really fast when I came back. Any comments/advice/tips on general code quality (and inevitable redundancy I'm sure) are always appreciated :)
Community feedback
- @adarshcodesPosted over 4 years ago
Hey! @Senatrius, you did great work on this project, it looks amazing👍 Let's see other things,
1.Design
- your design looks great, hover-effect, layout, positioning of images everything works fine.
- Responsiveness
- Responsiveness also works nicely.
- Code
- Code is well-structured
- Issues aroused
- Solve the HTML issue
Everything is nice, keep coding🐱💻.
3 - @VincenzoMarcovecchioPosted over 4 years ago
Good job man!
From what I can notice here maybe try to fix that
<nav></nav>
element, give it some align-self so that when you hover on the link it's not jumping too muchand try to make the telephone number and the email address on the footer links you can click
Keep practicing 👍 :)
2 - @mattstuddertPosted over 4 years ago
Awesome work on this challenge, Senatrius! Your code looks great! There's not much I'd change with your HTML and your Sass code is well-written and is generating clean CSS selectors with low specificity.
Also, it's been great seeing you give feedback to others. You've been giving some great advice 🙂
Keep up the great work!
1
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