Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

stray debug console.log in code (src/eventtarget.js)

#103

Looks like the trunk has an unintended debug console.log in place. All others in the source are commented out.

Around line 132 in src/eventtarget.js

        if(!event._preventDefault){
            //At this point I'm guessing that just HTMLEvents are concerned
            //with default behavior being executed in a browser but I could be
            //wrong as usual.  The goal is much more to filter at this point
            //what events have no need to be handled
            console.log('triggering default behavior for %s', event.type); // <---- HERE
            if(event.type in Envjs.defaultEventBehaviors){
                Envjs.defaultEventBehaviors[event.type](event);
            }

Reported by nickg · March 14th, 2010 @ 06:54 PM

State: resolved
Milestone: 1.2.x
Assigned to: Thatcher Thatcher

Activity

  1. nickg
    nickg

    I attempted a github pull request which may or may not work!
    it's a tester before more interesting changes.

    --nickg

    March 15th, 2010 @ 04:41 AM

  2. nickg
  3. Thatcher
    Thatcher

    yeah I found this too, it was a stray artifact of some work I'm doing to
    provide default html event behaviors. sorry! I'm going to push out this
    first before I pull in the rest of these patches.

    March 16th, 2010 @ 05:13 PM

  4. nickg
  5. Thatcher
    Thatcher

    thanks, my brain is skipping sometimes when multitasking. need more ram.

    March 17th, 2010 @ 04:31 PM

  6. Thatcher
    Thatcher
    • State changed from new to resolved
    • Milestone set to 1.2.x

    March 17th, 2010 @ 04:40 PM

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.