Looking to hire developers?
Hey Gyz, I have completed fylo-dark-theme challenge . I am hunker after your valuable feedback. Please suggest me where I need to improve. Your feedback is like a gem to me.
@alnahian2003
Posted
Hey Dear, Salahuddin!
This is another good creation of you and you're doing great. Keep Doing Your Good Works <3
Happy Coding
@mdsalahuddin2001
Posted
Take 💞
@alnahian2003
Posted
♥
@limtedtorni000
Posted
Hi MD your solution look great and responsive, just one thing i suggest to add background-size: contain; for the bg-curvy it will make it look fantastic.
Also, as your solution report suggests, it's considered a best practice to only use one <h1> tag per page and use less important heading tags for the rest of the headings on the page (for both SEO and semantic reasons). Doing so will clear up your solution report and make more sense in the context of the importance of the different headings on the page.
Happy Coding.
@mdsalahuddin2001
Posted
@limtedtorni000 Thank you so much, dear. I will hardly follow your suggestion
@RostykL
Posted
Hi, It looks nice but the block with the title "Access your files, anywhere" is a little bit higher then it should be.
@mdsalahuddin2001
Posted
@RostykL Thank you. I will improve it
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
