Investigate/fix incorrect zooming issue #9

Open
sgrebnov opened this Issue Apr 20, 2012 · 2 comments

Comments

Projects
None yet
1 participant
Owner

sgrebnov commented Apr 20, 2012

If we have very simple page with a little content on it (like /tests/basic/gallery/html/dialog-box.html) there is a gap between the bottom of the page and the bottom of the browser.

In previous theme versions this could be seen as #2 "White space beneath the appbar on blank page".

This seems to be because of incorrect page zooming. Du e to this we also have incorrect position for the fixed toolbar so we have to add some additional margin to have it positioned correctly.

Some additional info/experiments

  1. In desktop and mobile browsers except IE Mobile < body > element stretches vertically with all its content if it is shorter than browser height.
  2. IE mobile stretches only < body> element w/o its content. So if body background color is different from page content color we see the gap between bottom of the page and bottom of the browser.
  3. Height, min-height, position properties and meta tag viewport don’t affect this behavior.
  4. Even when we set fixed height for < html> element, that is the parent for < body>, body stretches to full browser height.
  5. Even when footer position is 0 (page height = screen height) footer is displayed incorrectly. This means that either js returns wrong value for screen height or web browser zooming doesn’t work properly.
Owner

sgrebnov commented Apr 20, 2012

To test on WP device or simulator - there is a red color body gap
http://jsbin.com/eqogun/

Page source
http://jsbin.com/eqogun/edit

Owner

sgrebnov commented May 7, 2012

The gap is not part of the body. This is something special.

@sgrebnov sgrebnov added a commit that referenced this issue May 7, 2012

@sgrebnov sgrebnov page min-height fix as per additional research for #9 Investigate/fix…
… incorrect zooming issue
cf716a6

@sgrebnov sgrebnov added a commit that referenced this issue May 7, 2012

@sgrebnov sgrebnov page min-height fix as per additional research for #9 Investigate/fix…
… incorrect zooming issue
f3504d3

@sgrebnov sgrebnov added a commit that referenced this issue May 7, 2012

@sgrebnov sgrebnov page min-height fix as per additional research for #9 Investigate/fix…
… incorrect zooming issue
db649c0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment