Permalink
Browse files

fix popup position in IE

Test Plan:
In any IE, trigger the login popup-

Busted in prod:
http://fbrell.com/

Fixed after this:
http://fbrell.com/?server=sb

DiffCamp Revision: 117797
Reviewed By: brent
CC: platform-diffs@lists, lshepard, brent, naitik, mattwkelly, selekman
Revert Plan:
OK
  • Loading branch information...
1 parent d154c82 commit 7122b971ed2cb1c39de54b39879ecec33f49ea7e @daaku daaku committed May 25, 2010
Showing with 2 additions and 2 deletions.
  1. +2 −2 src/core/ui.js
View
@@ -252,10 +252,10 @@ FB.provide('UIServer', {
: window.screenTop,
outerWidth = typeof window.outerWidth != 'undefined'
? window.outerWidth
- : document.body.clientWidth,
+ : document.documentElement.clientWidth,
outerHeight = typeof window.outerHeight != 'undefined'
? window.outerHeight
- : (document.body.clientHeight - 22),
+ : (document.documentElement.clientHeight - 22), // 22= IE toolbar height
width = call.size.width,
height = call.size.height,
left = parseInt(screenX + ((outerWidth - width) / 2), 10),

0 comments on commit 7122b97

Please sign in to comment.