Skip to content

##+js(addEventListener-logger.js) breaks page #368

Closed

Description

@mapx-
  • add wsj.com##+js(addEventListener-logger.js)
  • go to https://www.wsj.com/articles/wall-streets-big-landlords-are-so-hungry-for-houses-theyre-building-them-11546511401
  • the sliding strip at the top is broken (does not appear anymore)

chrome beta uBo last dev build
FF stable uBo last dev build

Notes (@gwarser)

https://github.com/uBlockOrigin/uAssets/blob/1b8ac96f9842d21c995adcbf1f6dd06e49964b10/filters/resources.txt#L1875

.toString() should be removed in addEventListener-logger.js (or callback checked for valid value)

@gorhill

Activity

  1. jspenguin2017 commented on Jan 7, 2019

    @jspenguin2017

    To convert a value to a string, String(value) should be used, it works for any value, even null and undefined.

    Note that String(value) returns a primitive, however, new String(value) returns a string object (which can cause problems).

  2. mapx- commented on Apr 6, 2019

    @mapx-
    ContributorAuthor

    Another case:

    https://www.bzbasel.ch/basel/basel-stadt/beat-oberlin-fordert-mehr-austausch-zwischen-den-fakultaeten-rduit-denken-hat-keinen-platz-134305206

    I added today a filter to defeat the anti adb stuff
    @@||az-cdn.ch/*ads.js$script,domain=bzbasel.ch

    However, I noted again the logger scriptlet breaking the page (see the infinite loading widget on the right side under the menu)
    bzbasel.ch##+js(addEventListener-logger.js)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workingfixedissue has been addressed

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions