search

oliverjam / workstream-website

The new Workstream X marketing site
6 stars 1 forks source link

Spacing around 4 boxes #45

Closed Jamchiller closed 7 years ago

Jamchiller commented 7 years ago

This seems to have changed recently? Maybe I just didn't notice it before. There's now a lot of space around that section for some reason.

Chrome: screen shot 2017-03-31 at 11 26 13

Firefox:

screen shot 2017-03-31 at 11 26 54

oliverjam commented 7 years ago

Shrink the screen: it's to make sure there's enough space for the longest possible paragraph that might appear in that section.

Hacky but that whole section is tbh

Jamchiller commented 7 years ago

Oh so it must be accomodating this:

screen shot 2017-03-31 at 11 59 59

Probably should add in another break point for tablet which fixes this but all good for the moment. Not too much of an issue

oliverjam commented 7 years ago

Tbh I'm really unhappy about this whole section. It's so so brittle it could break at any time on any device. Can't think of another way to do this design though

Jamchiller commented 7 years ago

Yeah well I guess we live with it for the moment!

I think a (slightly hacky) fix for th aesthetic issue is adding a teensy bit more spacing on the other sections? Especially because, if you look at the firefox version, the phone is super close to the line so looks out of whack. Adding in a bit more spacing might make it look better? Not much