Skip to content

Commit f2bcc3c

Browse files
authoredJan 21, 2025··
No longer request cookies if the user has refused (#61)
1 parent 4e8b97f commit f2bcc3c

File tree

2 files changed

+14
-5
lines changed

2 files changed

+14
-5
lines changed
 

‎docs/templates/js/functions.js

+12-4
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,16 @@
1-
let isCookiesAccepted = getCookie("cookies-accepted") === "true";
2-
31
// <> Cookies
2+
3+
// Returns whether the user has accepted the cookies or not, or undefined if the user hasn't chosen yet
4+
function isCookiesAccepted() {
5+
const cookie = getCookie("cookies-accepted");
6+
if (!cookie) {
7+
return undefined;
8+
}
9+
return Boolean(cookie);
10+
}
11+
412
function setCookie(cname, cvalue, exdays, force = false) {
5-
if (!isCookiesAccepted && !force)
13+
if (!isCookiesAccepted() && !force)
614
return;
715

816
const d = new Date();
@@ -37,7 +45,7 @@ function getCookie(cname) {
3745
* @param {double} exdays time in days
3846
*/
3947
function setStorageItem(item, value, exdays, force = false) {
40-
if (!isCookiesAccepted && !force)
48+
if (!isCookiesAccepted() && !force)
4149
return;
4250

4351
const d = new Date();

‎docs/templates/js/main.js

+2-1
Original file line numberDiff line numberDiff line change
@@ -410,7 +410,7 @@ if (examples) {
410410
// Example Collapse </>
411411

412412
// <> Cookies Accecpt
413-
if (!isCookiesAccepted) {
413+
if (isCookiesAccepted() === undefined) {
414414
document.body.insertAdjacentHTML('beforeend', `<div id="cookies-bar"> <p> We use cookies and local storage to enhance your browsing experience and store github related statistics. By clicking "Accept", you consent to our use of cookies and local storage. </p><div style="padding: 10px; white-space: nowrap;"> <button id="cookies-accept">Accept</button> <button id="cookies-deny">Deny</button> </div></div>`);
415415
}
416416

@@ -423,6 +423,7 @@ if (cookiesAccept && cookiesDeny) {
423423
cookiesBar.remove();
424424
});
425425
cookiesDeny.addEventListener('click', () => {
426+
setCookie('cookies-accepted', false, 99, true);
426427
cookiesBar.remove();
427428
});
428429
}

0 commit comments

Comments
 (0)
Please sign in to comment.