Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Could we replace the links back to normal? #24

Open
rolandschuetz opened this issue Jun 4, 2020 · 2 comments
Open

Could we replace the links back to normal? #24

rolandschuetz opened this issue Jun 4, 2020 · 2 comments

Comments

@rolandschuetz
Copy link

rolandschuetz commented Jun 4, 2020

In javascript we could add:

requestAnimationFrame(function() {
 Array.prototype.forEach.call(
    document.querySelectorAll('a[href^="javascript:linkTo_UnCryptMailto"]'),
    function(link) { link.innerHTML = link.innerHTML.replace(' (at) ', '@')}
  );
});
@hlubek
Copy link
Contributor

hlubek commented Jun 4, 2020

Good idea! Is there a reason why requestAnimationFrame should be used? I guess a dom ready event would be sufficient here.

We should put that behind an option though, I think there might be a number of crawlers with headless JS where that might disable the "protection"

@rolandschuetz
Copy link
Author

I added requestAnimationFrame only to improve the speed, the transformation can happen after an initial paint to not slow things down.

I have no ideas about modern crawlers, for headless js probably " (at) " is also very obvious. But we could add a 3sec delay.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants