3 Column Preview Card, VS Code, Google Chrome

Community feedback

• 

  Vanza Setia•27,775

  @vanzasetiaPosted over 3 years ago

  👋Hi Srijan1972! My name is Vanza!

  I have some feedback on this solution:

  • First, you don't need JavaScript for this challenge and also the way to link js is not like this:

  <link rel="script" href="main.js">
  

  • Instead you put your script tag right above the ending body tag. For some reason I can't show you the script tag here, but you can google html script tag.
  • To do mobile layout you can just set float: none or try to Google it first. But, there is a better way rather than use float. Try to use flexbox or grid, it will make this process much easier.
  • In this case, you don't need to set a height on your div, since setting height means not allow the default behavior, which is already responsive.

  That's it! Hopefully this is helpful!

  Marked as helpful

  0
• 

  Yashasvi Singh•890

  @aUnicornDevPosted over 3 years ago

  Hi Srijan,

  You have a good grasp of laying out... One better way of doing this is using flexbox. You could easily put them in single row in desktop and using a single statement can change it to single column multi row layout.

  0