Blogr landing page Responsive css/ add advanced dropdown menus

Community feedback

• 

  Dytoma•550

  @DytomaPosted almost 2 years ago

  Hey 👋

  Good job on completing this challenge, your solution looks amazing👌

  However I do have one suggestion when I looked at your code:

  • Instead of using a <div class="line"></div> I suggest you use border-bottom as the div add more content to your page, it is better to use borders.

  Happy coding👏

  Marked as helpful

  0

  ---

  

  Bakhriev•110

  @BakhrievPosted almost 2 years ago

  @Dytoma thanks for the feedback,in this example, should the background images be done with the img tag? or should they be done with the background?

  1

  

  Dytoma•550

  @DytomaPosted almost 2 years ago

  @Bakhriev you're welcome🙌

  For the call to action section you can use the background property to give it both the background color and the background image but for the other images, you can use the img element and wrap it in a div so that you can position it relative to the div.

  You can read more about backgrounds and how to set it on MDN multiple background docs.

  Marked as helpful

  0