Remove content-section class from <sections>

rhnvrm / breeze-website

:sparkles: Official Website for Breeze 2016 :sparkles:

http://rhnvrm.github.io/breeze-website

Apache License 2.0

4 stars 6 forks source link

Remove content-section class from <sections> #28

Closed FlameFractal closed 8 years ago

FlameFractal commented 8 years ago

Content-section class adds too much of padding which isn't looking good. Eg. when you click section names from the hamburger menu, they appear too far down and do not fit the page.

PFA screenshot when clicked "About Us". The arrows for the next section should preferably appear within this page.

screen shot 2016-07-03 at 11 25 44 am

ghost commented 8 years ago

@FlameFractal , I think there's no need for arrows above all sections...Doesn't look so good that way :) Instead what I suggest is-

Having a down-scroll on the first section and
Having an up-scroll on the last section That's it...

That would be pretty decent ! What say @rhnvrm @mr-karan @arafsheikh ?

rhnvrm commented 8 years ago

rather than removing the class, change grayscale.css line 233 to 100px padding. Without padding it looks bad.

FlameFractal commented 8 years ago

Dammit. I'm so stupid. I added 100vh to all the sections, so now all take up the entire screen and are well visible 👀 Can close this issue now. I will send a PR as soon as i fix the other issues, and merge some conflicts.