Apply BodySize

[ZheSun88]

This change is 

[CLAassistant]

All committers have signed the CLA.

[denis-anisimov]

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

---

Comments from Reviewable

[mvysny]

In order for the @BodySize annotation to be effective, you first need to remove the height: auto; applied to body from the styles.html file.

[ZheSun88]

@mvysny oh, Thanks for that.. I will remove that now.

from the browser console, in fact, the annotation will overright the setting in the styles.html. could you verify?

[denis-anisimov]

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

---

Comments from Reviewable

[mvysny]

Thanks @ZheSun88 ! Now the problem is that Beverage Buddy was designed with wrap height; changing that to 100vh requires redesign of all layouts... And that's quite a hell of a work. I'm trying to do that and I'm hitting some bugs - e.g. Grid being always taller than required.

[ZheSun88]

@mvysny from my side, everything still looks fine and all the layouts are still well designed .. maybe i should verify it with others..

[mvysny]

It looks fine, that is correct. However, if you notice the div (or rather vertical layout) containing the list of Reviews, it now overflows the body element. I don't think this is a good CSS practice, or is it?

[ZheSun88]

@mvysny Oh, i see the ugly horizontal scrollbar now.. Thanks for noticing that..I will try to fix it

[ZheSun88]

a separate ticket about the scrollbar is created. #147