
Solution retrospective
want to have easy coding.
What challenges did you encounter, and how did you overcome them?responsive.
What specific areas of your project would you like help with?responsive is not working.
Please log in to post a comment
Log in with GitHubCommunity feedback
- @kudos2Shef
@Bobu007 Congratulations on completing this challenge 👏. Your solution is great and responsive as well.
Few suggestions on your code:-
-
There is no need to use
flex
inbody
element. If you want to use it, apply to adiv
/section
instead. Within this div/section, you can write your .cards and .attribution classes. The flex in the body tag is causing your .attribution div to be displayed in a row instead of a column . -
In the .cards, add:
max-width:fit-content; margin:auto;
Use max-width along with min-width to make your code more efficient.
- Simply add
text-align:center;
in .attribution class . There is no need ofposition:absolute;
, as absolute positioning is used for layering elements on top of each other, which prevents proper text alignment. Instead, use relative, which keeps elements positioned one after another.
Hope this helps! Happy Coding
-
Join 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