Design comparison
Solution retrospective
I think I overwrite a lot of CSS, and should I just put the font size given in the body selector and that's it?.. because the sizes seems different.
Community feedback
- @correlucasPosted over 2 years ago
πHello GIGADEV, congratulations for your new solution!
π€ I was about to go to bed and switch the TV to watch Netflix, but then I saw your solution and I was wondering, I cannot let GIGA without a comment, otherwise some other nerd will write him to use rem units or to clean his CSS code and bla bla bla bla
So this time I don't have an advice for you, the solution is just perfect, even the double background you've set correctly, the media queries also... so you don't let a choice...πππ I created a media query in case someone opens your website in a tiny Nokia Phoneπ± with the screen of 240px and make the profile section change the flow vertically to display the profile photo in one row and the job title in the second row.
@media (max-width: 240px) { .review-content .info { display: flex; align-items: center; flex-direction: column; } }
π§ββοΈHope it helps you GIGA (I know that dont) But keep it up, you're getting better, faster and giga every challenge I seeπͺπͺπͺ
Marked as helpful1@Khalid-debuggPosted over 2 years ago@correlucas of course anything you type will help, you even care about Nokia users.. you are just awesome XD
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