All comments

• 

  Amoo Babatunde• 220

  @Khoded1

  Submitted 9 months ago

  hey guys, pls i'm still having some issues on background.. i could not fix the section 2 background.... pls help me out guys....

  All feedback are welcome!!!

  Thank you guys

  

  Fylo landing page with two column layout

  1

  6

  0

  

  Alessandra Oliveira• 690

  @itsale-o

  Posted 9 months ago

  Congratulations on completing the challenge!

  About your issue: you can put a <div> between the first and the second sections. Something like this:

  <div id="section1">
      your code here
  </div>
  <div id="bg-div"></div>
  <div id="section2>
      your code here
  </div>
  

  and then, on your CSS file you ajust the bg-div by putting the background-image and giving the div a height that fits the whole background image.

  Marked as helpful

  0

  This comment has received 1 replyShow
• 

  Oluwasanjo Elisha Adeniyi• 160

  @Sanjoclinton

  Submitted 9 months ago

  I had issues with placing my QR at the center of my code.

  

  QR-CODE Component challenge solution

  1

  9

  0

  

  Alessandra Oliveira• 690

  @itsale-o

  Posted 9 months ago

  Congratulations on completing the challenge! Nice work.

  About your issues: if you put the qr code image inside a div is easier to centralize

  You could do something like this:

  • On the HTML

  <div class=qr-code>
       <img src="qr-code">
  </div>
  

  • On the CSS

  .qr-code{
  	display: flex;
  	align-items: center;
  	justify-content: center;
  } 
  

  If you want, you can even put the texts on another div, and centralize it as well. It's easier to work with single divs, one at a time (talking from my personal experience)

  Marked as helpful

  0
• 

  SeanBailey15• 10

  @SeanBailey15

  Submitted 9 months ago

  Student here, looking for feedback on my CSS in particular.

  • Ways to make the CSS cleaner
  • Is there any redundant/useless code
  • Other general tips or critiques

  

  A Beginner Student's Attempt

  2

  7

  0

  

  Alessandra Oliveira• 690

  @itsale-o

  Posted 9 months ago

  Congratulations on completing this challenge!

  I'm not a master coder myself, but for me your code is pretty clean, and the solution looks very nice.

  The minimum change I'd make is to put the font-family on the body, then you don't need to say what's the font everytime, since it doesn't change. Of course is not necessary, it's just something I prefer to do

  Marked as helpful

  0

  This comment has received 1 replyShow
• 

  Faruque Dewan• 360

  @faruquedewan

  Submitted about 1 year ago

  I have two questions:

  1. Is it a good practice to use different CSS files for different screen widths? If not then how to design mobile CSS after designing Desktop CSS? (Some things are overridden.)
  2. How to get the Icons in center? (They are popping up a bit.)

  

  Results Summary Component | Mobile First | Flexbox

  4

  5

  0

  

  Alessandra Oliveira• 690

  @itsale-o

  Posted about 1 year ago

  About the first question: I use one CSS file for both. I did

  • @media(max-width: 1400px) {here I did the desktop deseign} and then
  • @media(max-width: 700px){here I did the mobile design}

  This usually works.

  About the icons: if I get the question right, if you use align-items: center, it may solve the problem

  0

  This comment has received 4 repliesShow