Latest solutions
rest-countries-api-with-color-theme-switcher-master
#fetch#react#tailwind-cssSubmitted over 2 years ago
Latest comments
- @Tony-dopeSubmitted over 2 years ago@Har1s-AkbarPosted over 2 years ago
Nice try there are few things i would like to point out in your mobile version. It seems you are using max width for the text on mobile version too, you should use 60%-50% width on text with m-auto to center it. if you want to center it without reducing the width you should use text center css property. I would suggest you decrease the text-size, text width on mobile version with text center. I would also suggest you to use tailwind css because it rids you of trouble of writing the css in separate file.
0 - @gmathankumarSubmitted over 2 years ago@Har1s-AkbarPosted over 2 years ago
Well done you did a great job but there are some issues I see in the design.
- In mobile version I suppose you are using height screen which is not a best practice, it should be height full. 2)In desktop version you should use width full in that way your design will span across whole screen.
Marked as helpful0 - @RomWeb3Submitted over 2 years ago@Har1s-AkbarPosted over 2 years ago
You have done a great job. There are somethings wrong with the mobile version, I think you are using height to the screen because it is not scrollable on mobile version and also the background is not covering the whole width of screen on desktop version. You should fix these issues. All in all you did a great job, keep coding
0 - @Har1s-AkbarSubmitted over 2 years ago@Har1s-AkbarPosted over 2 years ago
I figured out the way to add the transition thanks everyone
0 - @himym413Submitted over 2 years ago@Har1s-AkbarPosted over 2 years ago
Good Job, well done I think there are some of the things you might have done better 1)Like the divider is a bit large it is drawing more attention than necessary and the font of advice is also a bit large.
All in all well done
1