4 ms·
Thanks for pointing that out, I think I might remove it. Will cause more trouble down the road. Do you have any ideas to make it more safe?
by thomaswangio 7y ago
Thanks for pointing that out, I think I might remove it. Will cause more trouble down the road. Do you have any ideas to make it more safe?
- supernintendo 7y agoUnfortunately, I can’t think of any. Twitter could arbitrarily change where those elements are placed in the DOM at any time (and they probably will to prevent other extensions from blocking promoted tweets). I would just consider this out of the scope of your project and not bother handling it.
- thomaswangio 7y agoTotally agreed, definitely out of scope. Just removed the script from the manifests. Thanks again!
- djyaz1200 7y agoThomas, I like your work. Just joined your newsletter. Would be interested to talk. I'm @ www.sendsmart.com
- thomaswangio 7y agoSure, I'm at thomas [at] wang.sh!
- daxterspeed 7y agoYou could potentially keep searching upwards for a parentNode that has role="article". While Twitter could change their markup significantly they sorta have to support screen readers so aria attributes will poke through here and there.
- thomaswangio 7y agoHmmm interesting, if you have some time I'd love a PR. It would be a nice feature to have tbh