Skip to content

Commit

Permalink
fixed a couple test failures
Browse files Browse the repository at this point in the history
git-svn-id: http://svn.mochikit.com/mochikit/trunk@736 14cb8549-11fe-0310-83a5-45f3e82da484
  • Loading branch information
Beau Hartshorne committed Apr 25, 2006
1 parent 582fbb5 commit a818055
Show file tree
Hide file tree
Showing 4 changed files with 11 additions and 6 deletions.
11 changes: 7 additions & 4 deletions MochiKit/DOM.js
Expand Up @@ -272,16 +272,19 @@ MochiKit.Base.update(MochiKit.DOM, {


/* /*
Opera & (Safari absolute) incorrectly account for body Opera < 9 and old Safari (absolute) incorrectly account for
offsetTop and offsetLeft. body offsetTop and offsetLeft.
*/ */
var ua = navigator.userAgent.toLowerCase(); var ua = navigator.userAgent.toLowerCase();
if (ua.indexOf('opera') != -1 || if ((typeof(opera) != "undefined" &&
parseFloat(opera.version()) < 9) ||
(ua.indexOf('safari') != -1 && (ua.indexOf('safari') != -1 &&
self.computedStyle(elem, 'position') == 'absolute')) { self.computedStyle(elem, 'position') == 'absolute')) {

c.x -= b.offsetLeft; c.x -= b.offsetLeft;
c.y -= b.offsetTop; c.y -= b.offsetTop;

} }
} }


Expand Down
1 change: 1 addition & 0 deletions MochiKit/Signal.js
Expand Up @@ -657,5 +657,6 @@ MochiKit.Signal.__new__(this);
connect = MochiKit.Signal.connect; connect = MochiKit.Signal.connect;
disconnect = MochiKit.Signal.disconnect; disconnect = MochiKit.Signal.disconnect;
disconnectAll = MochiKit.Signal.disconnectAll; disconnectAll = MochiKit.Signal.disconnectAll;
signal = MochiKit.Signal.signal;


MochiKit.Base._exportSymbols(this, MochiKit.Signal); MochiKit.Base._exportSymbols(this, MochiKit.Signal);
3 changes: 2 additions & 1 deletion packed/MochiKit/MochiKit.js
Expand Up @@ -2612,7 +2612,7 @@ _330=_330.offsetParent;
} }
} }
var ua=navigator.userAgent.toLowerCase(); var ua=navigator.userAgent.toLowerCase();
if(ua.indexOf("opera")!=-1||(ua.indexOf("safari")!=-1&&self.computedStyle(elem,"position")=="absolute")){ if((typeof (opera)!="undefined"&&parseFloat(opera.version())<9)||(ua.indexOf("safari")!=-1&&self.computedStyle(elem,"position")=="absolute")){
c.x-=b.offsetLeft; c.x-=b.offsetLeft;
c.y-=b.offsetTop; c.y-=b.offsetTop;
} }
Expand Down Expand Up @@ -4376,6 +4376,7 @@ MochiKit.Signal.__new__(this);
connect=MochiKit.Signal.connect; connect=MochiKit.Signal.connect;
disconnect=MochiKit.Signal.disconnect; disconnect=MochiKit.Signal.disconnect;
disconnectAll=MochiKit.Signal.disconnectAll; disconnectAll=MochiKit.Signal.disconnectAll;
signal=MochiKit.Signal.signal;
MochiKit.Base._exportSymbols(this,MochiKit.Signal); MochiKit.Base._exportSymbols(this,MochiKit.Signal);
if(typeof (dojo)!="undefined"){ if(typeof (dojo)!="undefined"){
dojo.provide("MochiKit.Visual"); dojo.provide("MochiKit.Visual");
Expand Down
2 changes: 1 addition & 1 deletion tests/test_MochiKit-DOM.html
Expand Up @@ -7,7 +7,7 @@
<script type="text/javascript" src="SimpleTest/SimpleTest.js"></script> <script type="text/javascript" src="SimpleTest/SimpleTest.js"></script>
<link rel="stylesheet" type="text/css" href="SimpleTest/test.css"> <link rel="stylesheet" type="text/css" href="SimpleTest/test.css">
</head> </head>
<body style="margin: 10px; padding: 10px;"> <body>


<div id="posTest" style="position: absolute; left: 400px; top: 100px; width: 100px; height: 100px; background: red;">&nbsp;</div> <div id="posTest" style="position: absolute; left: 400px; top: 100px; width: 100px; height: 100px; background: red;">&nbsp;</div>


Expand Down

0 comments on commit a818055

Please sign in to comment.