Design comparison
Solution retrospective
Hello 👋,
This is my solution to the frontendmentor challenge. I was mainly using flexbox to achieve the layout. Any comment and feedback are much appreciated.
Till then, keep on coding 👏🎉
Community feedback
- @FluffyKasPosted about 3 years ago
Hey, nice looking solution!
There's a few small things you could improve on though. Your headings should start from h1. You also seem to reference, h1, h2 and h3 in your css but in your html I can only see one h2. You could also give a
background-size
to your background images so they fit the screen ^^0@agusthasPosted about 3 years ago@FluffyKas Thank you 👏
Yeah i was using h1 before but then i changed it to h2 because i was thinking if this was to be a component, there should be an h1 up there, also thats why i reference h1, h2, and h3 in the css.
And i forgot about that
background-size
cover, was planning to put it there (maybe i should've make a todo list before submitting) 😄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