Your feedbacks will be appreciated. thanks
Mohireza
@Mohireza1All comments
- @d3vaqaSubmitted over 2 years ago@Mohireza1Posted over 2 years ago
So uh, did you make a mobile layout too? Because I get the desktop layout when I open your preview with my phone.
If you have, then consider merging both layouts in one single code base and if you haven't, then just make a mobile layout because that's a part of the challenge.
Marked as helpful0 - @jesuisbienbienSubmitted over 2 years ago
This challenge was a little tough for me. I have a few questions:
-
How can I make my code more clean and efficient?
-
I use rem and px mixed up, it seems unorganized for me. Any suggestions?
-
I used border-top-left-radius, border-top-right-radius, etc... Because in this challenge, the rounded corners change in mobile and desktop view. Better way to set the border-radius?
Any other feedbacks and suggestions are welcome as well. I'd love to learn. Thank you! :D
@Mohireza1Posted over 2 years agoHey, I just wanted to thank you for this great solution! I learned a lot from your code, and I feel like it's just the refined version of every solution here. Thank you!
1 -
- @twoicedstoneSubmitted over 2 years ago
I need help with the mobile view layout.
@Mohireza1Posted over 2 years agoBefore solving these challenges, you need to be familiar with both flexbox and grids. I suggest that you learn flexbox using Flexbox Froggy and then maybe use flex for centering stuff in your pages.
You can also learn about girds (the people behind Froggy have a grids tutorial, too), which are, in my opinion, essential to designing any webpage.
1