refactor(position): Remove deprecated body scrollTop and scrollLeft

[mvhecke]

Removes the use of deprecated variables in strict mode which caused the following errors(#1658).

body.scrollTop is deprecated in strict mode. Please use 'documentElement.scrollTop' if in strict mode and 'body.scrollTop' only if in quirks mode.
body.scrollLeft is deprecated in strict mode. Please use 'documentElement.scrollLeft' if in strict mode and 'body.scrollLeft' only if in quirks mode.

[mvhecke]

It does pass the build test, I'm however not sure about the behavior in all supported browsers.

[pkozlowski-opensource]

@Gamemaniak you can use this little test file:
https://github.com/angular-ui/bootstrap/blob/master/src/position/test/test.html

to manually / visually check if things are allright in all the supported browsers.

[mvhecke]

@pkozlowski-opensource I'll check it out tomorrow.

• Chrome 32
• Firefox 26
• Opera 12
• Safari
• IE 11
• IE 10
• IE 9

[mvhecke]

It'll be a day later, haven't got enough time today.

[mvhecke]

@pkozlowski-opensource I've tested it and it seems fine, although I tested it in IE11 with developer mode for IE9 and IE10 so I'm not sure if that representative.

[bekos]

@Gamemaniak You can try this: http://www.modern.ie/ :D

[ajoslin]

@Gamemaniak I accidentally deleted your latest comment with the delete button - sorry :) You might want to repost it

[mvhecke]

No problem @ajoslin. The only browser that was giving me strange behavior was IE10, but that was due to opening the developer tools which triggered IE7 mode. But positioning seems fine in all the tested browsers with this fix.

[pkozlowski-opensource]

Cool, I think this one is good to go in. In fact those deprecated calls were introduced for IE8 in 9aecd4e. But since we don't support IE8 we don't care, I guess.

[pkozlowski-opensource]

Thnx @Gamemaniak !