Permalink
Browse files

fix failing offset tests

  • Loading branch information...
mikesherov committed Oct 17, 2012
1 parent a151471 commit fad0e221818f0a78550584b3c0a16a6808ecd921
Showing with 19 additions and 2 deletions.
  1. +19 −2 test/unit/offset.js
View
@@ -295,11 +295,26 @@ testIframe("offset/static", "static", function( $ ) {
});
testIframe("offset/fixed", "fixed", function( $ ) {
+ // IE is collapsing the top margin of 1px; detect and adjust accordingly
+ var ie = $("#fixed-1").position().top === 2;
+
expect(34);
var tests = [
- { "id": "#fixed-1", "offsetTop": 1001, "offsetLeft": 1001, "positionTop": 0, "positionLeft": 0 },
- { "id": "#fixed-2", "offsetTop": 1021, "offsetLeft": 1021, "positionTop": 20, "positionLeft": 20 }
+ {
+ "id": "#fixed-1",
+ "offsetTop": 1001,
+ "offsetLeft": 1001,
+ "positionTop": ie ? 2 : 0,
+ "positionLeft": ie ? 2 : 0
+ },
+ {
+ "id": "#fixed-2",
+ "offsetTop": 1021,
+ "offsetLeft": 1021,
+ "positionTop": ie ? 22 : 20,
+ "positionLeft": ie ? 22 : 20
+ }
];
jQuery.each( tests, function() {
@@ -318,6 +333,8 @@ testIframe("offset/fixed", "fixed", function( $ ) {
// need to have same number of assertions
ok( true, "Fixed position is not supported" );
ok( true, "Fixed position is not supported" );
+ ok( true, "Fixed position is not supported" );
+ ok( true, "Fixed position is not supported" );
}
});

0 comments on commit fad0e22

Please sign in to comment.