Responsive landing page with form validation

Community feedback

• 

  Ahmed Faisal•5,095

  @afrusselPosted over 3 years ago

  I didn't know why there are 4 css files use for this project. More over use these 4 files after Google font. Other then that it's good

  Marked as helpful

  1

  ---

  

  Leon Michalak•545

  @NinjaInShadePosted over 3 years ago

  @afrussel There are 3 css files as far as I know. One for button/s, a main css file where the bulk of styling is, and a base for base styles, overwrites etc...

  It's good practice not to chuck everything in one file.

  0
• 

  P

  Patrick•14,265

  @palgrammingPosted over 3 years ago

  You should remove the background color out of your main and put it on the body tag

  0

  ---

  

  Leon Michalak•545

  @NinjaInShadePosted over 3 years ago

  @palgramming I wanted to make sure the bg was only used for the height of the content. If it was body, and you had a really big monitor, it'd look weird

  0

  

  P

  Patrick•14,265

  @palgrammingPosted over 3 years ago

  @NinjaInShade No adding a background color to your body tag is not going to make anything look weird but is going to get rid of the weird white gap at the bottom of your page

  0

  

  Leon Michalak•545

  @NinjaInShadePosted over 3 years ago

  @palgramming That's what I meant. I was being as pixel-perfect to the design as possible, and the design has a certain height which more or less mine is too. I guess if im not being perfect about it I should do this. Thanks, i'll be doing it :)

  0