Alex-D / Cookies-EU-banner

1kb vanilla JS script which manages cookies consent banner display like asked by GDPR
http://alex-d.github.io/Cookies-EU-banner/
MIT License
439 stars 57 forks source link

Does not disable banner for Google bot #17

Closed jenstornell closed 8 years ago

jenstornell commented 8 years ago

Disables banner when visitor is a bot : prevents SEO Engines to confuse your cookie advert message with the main content of your pages;

I tried https://developers.google.com/speed/pagespeed/insights/ today and the above does not seems to be correct.

Look below:

pagespeed insights

It shows the banner and it uses the js to do it because I set display to none, just like your example. Just wanted to let you know that the banner seems to be crawlable after all.

Alex-D commented 8 years ago

https://github.com/Alex-D/Cookies-EU-banner/blob/master/src/cookies-eu-banner.js#L20

There is a new user agent maybe :/

Alex-D commented 8 years ago

I've made a test. PageSpeed use a classic browser and is not exactly a bot since you trigger it.