This project is archived and is in readonly mode.

#508 open
Deleted User

Bug with getComputedSize and Ipad

Reported by Deleted User | February 4th, 2011 @ 10:00 AM | in 1.4.0.1 (closed)

There is a problem with getComputedSize function (tested on mootools-more 1.2+) on Ipad device.

When the function is called like this : getComputedSize({'styles':['padding','margin','border','scroll']});

the style "scroll-top "return null and when the function try to do ".toInt()" on this value, its failed.

Try it : http://jsfiddle.net/neilime/PWcct/

Comments and changes to this ticket

  • Arian

    Arian February 8th, 2011 @ 12:29 AM

    Could you also test with MooTools 1.3 (although the jsfiddle uses 1.3).

    Also I don't know if scroll-top is a valid css property you can calculate with getComputedSize. It also fails in Chrome.

  • Deleted User

    Deleted User February 8th, 2011 @ 07:41 AM

    "Could you also test with MooTools 1.3" i've tested it : "tested on mootools-more 1.2+";

    "Also I don't know if scroll-top is a valid css property" this is getComputedSize that calculates scroll-top.

    Here is the patch i made for mootools 1.3 :

    Element.getComputedSize :

    getStylesList(options.styles, options.planes).each(function(style){

    styles[style] = (this.getStyle(style) !== null)?this.getStyle(style).toInt():0;
    

    }, this);

  • Arian

    Arian February 8th, 2011 @ 09:08 AM

    • State changed from “new” to “open”
    • Milestone set to 1.3.1.1
    • Assigned user set to “Arian”
    • Milestone order changed from “197561” to “0”

    We can do that, but scroll or scroll-top still isn't a valid css property, which getComputedSize only accepts. However I do think it shouldn't throw errors.

  • Arian

    Arian February 25th, 2011 @ 08:15 PM

    • Milestone changed from 1.3.1.1 to 1.4.0.1
    • Milestone order changed from “52” to “0”

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

The MooTools Extensions

People watching this ticket

Pages