Design comparison
Solution retrospective
I could not figure out the JS part
Community feedback
- @correlucasPosted about 2 years ago
👾Hello @Inioluwa2003, Congratulations on completing this challenge!
Great solution and a great start! From what I saw you’re on the right track. I’ve few suggestions for you that you can consider adding to your code:
Remove the
width: 1440px
to align, this is only a reference for the design, see the code for the alignment below:body { min-height: 100vh; font-size: 15px; background-color: hsl(216, 12%, 8%); /* max-width: 1440px; */ display: flex; align-items: center; justify-content: center; flex-direction: column; }
Use flex-wrap to make the button adjust automatically with the container size:
.rates { display: flex; flex-wrap: wrap; }
When you download the project files there’s a file called
style-guide.md
where you can find information such ashsl color codes
and thefont-size
for the headings.✌️ I hope this helps you and happy coding!
Marked as helpful0 - @faha1999Posted about 2 years ago
Hello, Inioluwa Adeyemo Congratulations on finishing this project. It's lovely and great on the whole! Just a little tip:
- add the below code to the body. It will center everything
body { justify-content: center; align-items: center; display: flex; height: 100vh; flex-direction: column; }
-
remove
margin
from.section
. it's not necessary anymore. -
** do the same thing on the 2nd page**
-
add
alt="icon"
attribute inimg
. This would help improve accessibility.
I hope it will work. Happy coding.
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