Changeset 495

Show
Ignore:
Timestamp:
01/14/06 02:11:06 (3 years ago)
Author:
therve@gmail.com
Message:

Use elementDimensions

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • mochikit/branches/scriptaculous/MochiKit/Effects.js

    r494 r495  
    649649    element = MochiKit.DOM.getElement(element); 
    650650    var oldHeight = MochiKit.DOM.getStyle(element, 'height'); 
    651     var elementDimensions = MochiKit.DOM.getDimensions(element); 
     651    var elementDimensions = MochiKit.DOM.elementDimensions(element); 
    652652    options = MochiKit.Base.update({ 
    653653        scaleContent: false, 
    654654        scaleX: false, 
    655655        scaleFrom: 0, 
    656         scaleMode: {originalHeight: elementDimensions.height, 
    657                     originalWidth: elementDimensions.width}, 
     656        scaleMode: {originalHeight: elementDimensions.h, 
     657                    originalWidth: elementDimensions.w}, 
    658658        restoreAfterFinish: true, 
    659659        afterSetup: function (effect) { 
    …… 
    752752    // SlideDown need to have the content of the element wrapped in a container element with fixed height! 
    753753    var oldInnerBottom = MochiKit.DOM.getStyle(element.firstChild, 'bottom'); 
    754     var elementDimensions = MochiKit.DOM.getDimensions(element); 
     754    var elementDimensions = MochiKit.DOM.elementDimensions(element); 
    755755    options = MochiKit.Base.update({ 
    756756        scaleContent: false, 
    757757        scaleX: false, 
    758758        scaleFrom: 0, 
    759         scaleMode: {originalHeight: elementDimensions.height, 
    760                     originalWidth: elementDimensions.width}, 
     759        scaleMode: {originalHeight: elementDimensions.h, 
     760                    originalWidth: elementDimensions.w}, 
    761761        restoreAfterFinish: true, 
    762762        afterSetup: function (effect) { 
    …… 
    848848        opacity: MochiKit.DOM.getInlineOpacity(element) }; 
    849849 
    850     var dims = MochiKit.DOM.getDimensions(element); 
     850    var dims = MochiKit.DOM.elementDimensions(element); 
    851851    var initialMoveX, initialMoveY; 
    852852    var moveX, moveY; 
    …… 
    857857            break; 
    858858        case 'top-right': 
    859             initialMoveX = dims.width; 
     859            initialMoveX = dims.w; 
    860860            initialMoveY = moveY = 0; 
    861             moveX = -dims.width; 
     861            moveX = -dims.w; 
    862862            break; 
    863863        case 'bottom-left': 
    864864            initialMoveX = moveX = 0; 
    865             initialMoveY = dims.height; 
    866             moveY = -dims.height; 
     865            initialMoveY = dims.h; 
     866            moveY = -dims.h; 
    867867            break; 
    868868        case 'bottom-right': 
    869             initialMoveX = dims.width; 
    870             initialMoveY = dims.height; 
    871             moveX = -dims.width; 
    872             moveY = -dims.height; 
     869            initialMoveX = dims.w; 
     870            initialMoveY = dims.h; 
     871            moveX = -dims.w; 
     872            moveY = -dims.h; 
    873873            break; 
    874874        case 'center': 
    875             initialMoveX = dims.width / 2; 
    876             initialMoveY = dims.height / 2; 
    877             moveX = -dims.width / 2; 
    878             moveY = -dims.height / 2; 
     875            initialMoveX = dims.w / 2; 
     876            initialMoveY = dims.h / 2; 
     877            moveX = -dims.w / 2; 
     878            moveY = -dims.h / 2; 
    879879            break; 
    880880    } 
    …… 
    910910                 }), 
    911911                 new Effect.Scale(effect.element, 100, { 
    912                         scaleMode: {originalHeight: dims.height, 
    913                                     originalWidth: dims.width}, 
     912                        scaleMode: {originalHeight: dims.h, 
     913                                    originalWidth: dims.w}, 
    914914                        sync: true, 
    915915                        scaleFrom: MochiKit.Base.isOpera() ? 1 : 0, 
    …… 
    938938        opacity: MochiKit.DOM.getInlineOpacity(element) }; 
    939939 
    940     var dims = MochiKit.DOM.getDimensions(element); 
     940    var dims = MochiKit.DOM.elementDimensions(element); 
    941941    var moveX, moveY; 
    942942 
    …… 
    946946            break; 
    947947        case 'top-right': 
    948             moveX = dims.width; 
     948            moveX = dims.w; 
    949949            moveY = 0; 
    950950            break; 
    951951        case 'bottom-left': 
    952952            moveX = 0; 
    953             moveY = dims.height; 
     953            moveY = dims.h; 
    954954            break; 
    955955        case 'bottom-right': 
    956             moveX = dims.width; 
    957             moveY = dims.height; 
     956            moveX = dims.w; 
     957            moveY = dims.h; 
    958958            break; 
    959959        case 'center': 
    960             moveX = dims.width / 2; 
    961             moveY = dims.height / 2; 
     960            moveX = dims.w / 2; 
     961            moveY = dims.h / 2; 
    962962            break; 
    963963    } 

  • mochikit/branches/scriptaculous/MochiKit/New.js

    r494 r495  
    227227    }, 
    228228 
    229     getDimensions: function (element) { 
    230         element = MochiKit.DOM.getElement(element); 
    231         if (MochiKit.DOM.getStyle(element, 'display') != 'none') { 
    232             return {width: element.offsetWidth, height: element.offsetHeight}; 
    233         } 
    234  
    235         // All *Width and *Height properties give 0 on elements with display none, 
    236         // so enable the element temporarily 
    237         var els = element.style; 
    238         var originalVisibility = els.visibility; 
    239         var originalPosition = els.position; 
    240         els.visibility = 'hidden'; 
    241         els.position = 'absolute'; 
    242         els.display = ''; 
    243         var originalWidth = element.clientWidth; 
    244         var originalHeight = element.clientHeight; 
    245         els.display = 'none'; 
    246         els.position = originalPosition; 
    247         els.visibility = originalVisibility; 
    248         return {width: originalWidth, height: originalHeight}; 
    249     }, 
    250  
    251229    makePositioned: function (element) { 
    252230        element = MochiKit.DOM.getElement(element);