All comments

• 

  Shadib Shah•610

  @shadib0797Submitted about 3 years ago

  Any Suggestions are much appreciated.

  

  Rest Countries API with Color Theme Switcher using HTML, CSS and JS

  2

  20
  

  Thomas•100

  @tamorosoPosted about 3 years ago

  Hi Shadib, great job, you nailed it ! There is plenty of html issues. Lots of them comes from the fact that you forgot to put alt attribute to your images. So you should add it to remove html issues.

  Keep coding 🌟

  0
• 

  Shreya Choure•45

  @shreyachoureSubmitted over 3 years ago

  For the desktop view, image is not occupying the whole div element. Please suggest which property I can use for it. Please review the code, your feedback and suggestions are valuable for me!

  

  Stats preview card component

  2

  20
  

  Thomas•100

  @tamorosoPosted over 3 years ago

  You could overcome your problem by importing your image as a background instead of putting it in your html file. You can then specify that your text div will take 50% of the card width and the other 50% is for the image. I've done the challenge and proceeded like this. I will upload it soon so you can see what i'm talking about.

  0