#1191 new
Terry Johnson

viewportOffset - introduced redundant code in 1.7.0.0

Reported by Terry Johnson | January 3rd, 2011 @ 03:53 PM

The element extension line here is new since 1.7_rc3 and does not appear to wrap the correct variable:

  function viewportOffset(forElement) {
    element = $(element);
    var valueT = 0, valueL = 0, docBody = document.body;

    var element = forElement;

Suggested fix:

  function viewportOffset(forElement) {
    var valueT = 0, valueL = 0, docBody = document.body;
    var element = $(forElement);

Comments and changes to this ticket

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

New-ticket Create new ticket

Create your profile

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

The Prototype JavaScript library.

Shared Ticket Bins

Pages