Yes, this is true, but doing it this way will slow down dragging actions (with scripe.aculo.us for ex.).
If I'm not wrong it's better to use : valueT += parseInt(element.style.borderTopWidth) || 0; valueL += parseInt(element.style.borderLeftWidth) || 0; cheers, Christophe. On 4 mar, 22:42, "Searle" <[EMAIL PROTECTED]> wrote: > I can't be bothered to find out how to open a bug report, so I'll just > put it here. Maybe somone cares. > > cumulativeOffset ignores borders of parent elements, which is wrong. > It was hard to find any information because ALL implementations I > found were wrong but one. (Astonishing, it's very basic.) > cumulativeOffset should read something like this: > > cumulativeOffset: function(element) { > var valueT = 0, valueL = 0; > do { > valueT += element.offsetTop || 0; > valueL += element.offsetLeft || 0; > > + valueT += Element.getStyle(element, 'borderTopWidth') || 0; > + valueL += Element.getStyle(element, 'borderLeftWidth') || 0; > > element = element.offsetParent; > } while (element); > return [valueL, valueT]; > }, > > Sorry, untestet. > > Cheers, > Searle --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Prototype: Core" group. To post to this group, send email to prototype-core@googlegroups.com To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/prototype-core?hl=en -~----------~----~----~----~------~----~------~--~---