-
Notifications
You must be signed in to change notification settings - Fork 51
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
Add cookie notice #146
Add cookie notice #146
Conversation
margin-left: 2em; | ||
white-space: nowrap; | ||
vertical-align: middle; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is Leap theme banner on RtD theme. It just doesn't feel right. 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Btw, to use something closer to RtD, it's enough to change the accept button. A few lines:
- font-size: 14px;
+ font-size: 16px;
- background: radial-gradient(circle, #4C71C6 0%, #002B86 100%);
+ background: rgb(41, 128, 185);
- text-transform: uppercase;
- letter-spacing: 1.2px;
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will change, thanks
$(document).ready(function () { | ||
var cookieDiv = document.createElement('div') | ||
cookieDiv.id = 'cookie-notice' | ||
cookieDiv.innerHTML = "<p style='margin: 1em'>Privacy and Cookies: This site uses cookies. By continuing to use this website, you agree to their use. To find out more, see our <a style='color:#4c71c6; text-decoration: underline;' href='https://cloud.dwavesys.com/leap/legal/privacy_policy/' target='_blank'>Privacy Policy</a>.<button id='cookie-accept' class='cookie-button'>Close and Accept</button></p>" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We're actually using Google Analytics, and Google is storing the cookie, not DWave.
The privacy policy (used on Leap) does mention sharing of user's data with 3rd party services that act on our behalf, but should here actually Google ask for user's permission, and say they'll share the analytics with DWave?
@randomir, I think this can be merged now, thanks. |
@randomir, can you take a quick look so I can do the same for dwave_tabu, thanks!