720kb / angular-tooltips

Angularjs tooltips module, add tooltips to your elements - https://720kb.github.io/angular-tooltips
351 stars 157 forks source link

Fix of positioning in IE #164

Closed tomecko closed 8 years ago

tomecko commented 8 years ago

Primarily this PR fixes the bug affecting IE9-IE11 tooltip positioning when tooltipAppendToBody attribute is passed. Basically top and left values have been based on window.scrollY and window.scrollX respectively. As these two values are not available in IE (resulting in misplaced toolips), pageYOffset and pageXOffset aliases should be used instead (see Notes in https://developer.mozilla.org/en-US/docs/Web/API/Window/scrollY).

By the way, I fixed some trivial stylesheet issues (just looking at scss lint output).

45kb commented 8 years ago

Looks great @tomecko thanks and thanks for linting :)