Responsive landing page using CSS Grid and Flex

Community feedback

• 

  MikevPeeren•2,100

  @MikevPeerenPosted about 3 years ago

  Hey @erictan86,

  Good job, try to keep larger desktop version in mind, on larger screens than 1440px the space between the bottom of the component and the text is a bit large.

  As for the color you can use the following, this however means that you need to load the image from css rather than using <img>

  background-color: #aa5cdb;
  background-blend-mode: multiply;
  

  Marked as helpful

  0

  ---

  

  Eric Tan•80

  @erictan86Posted about 3 years ago

  @MikevPeeren Load the image from CSS meaning I create a div in html without img? Then specify the image inside CSS?

  The space between the bottom of the component and text is too large - could you elaborate more on this portion?

  Thanks very much for your input. I really need it to improve.

  0

  

  MikevPeeren•2,100

  @MikevPeerenPosted about 3 years ago

  @erictan86 Yes, make a definition in css and then use it on a div for example.

  As for the text see this img: https://i.imgur.com/se7cGBQ.png I am talking about the empty space where no text is, this is a bit too much.

  0

  

  Eric Tan•80

  @erictan86Posted about 3 years ago

  @MikevPeeren got it. I'll take note. Thanks again.

  0
• 

  Yaungni•100

  @Yaungni-Linn-LattPosted about 3 years ago

  Greeting. As for me, I create a div container for the image. Then I apply background color to the container. Like this "background-color: hsl(277, 64%, 61%);". "61%" is the opacity of the background color.

  0

  ---

  

  Eric Tan•80

  @erictan86Posted about 3 years ago

  @Yaungni-Linn-Latt thanks. Let me give it a shot.

  0