@Suraj1333Submitted over 3 years ago
Any feedback would be helpful.
RITESH KUMAR
@RITESH29-webAll comments
- @Suraj1333Submitted over 3 years ago
Any feedback would be helpful.
@RITESH29-webPosted over 3 years agoHi Suraj, The preview is good. There are some minor problems that you can look at.
-
You can use flexbox property in main section for your .box class. And with it you can use flex-wrap property which will help to align the tabs.
-
You can also use transform: translateY(); property to align two tabs to sides.
If you have problem in understanding what I said or have more doubts you can check my solution.
0 -
- @sneha-debugSubmitted over 3 years ago
Please have a look at this and do let me know mistakes that I have done....
@RITESH29-webPosted over 3 years agoHi Sneha, Preview looks good. There are some minor points that you have left.....
-
In @margin you need to increase the max-width. As if now the transition feels smooth but the tabs resizing is too much.
-
In the <h2> sections of you add font-weight of 400.
-
In the *{} you can add box-sizing: border box.
If you have difficulty in understanding what I said, you can have a look at my solution for this project.
1 -
- @DanielstrtrSubmitted over 3 years ago
How do I make it more responsive? I thought that just converting every px to rem.
@RITESH29-webPosted over 3 years agoHi Daniel, I would suggest you add @media query with min-width or max-width values, it will give smooth transition for screen resizing.(I am also new to this and I am doing the same to update my project).
1
