Skip to content

botstopper: Add info about cookies #497

Closed
tdgroot wants to merge 3 commits into
masterfrom
botstopper_cookie
Closed

botstopper: Add info about cookies #497
tdgroot wants to merge 3 commits into
masterfrom
botstopper_cookie

Conversation

@tdgroot

@tdgroot tdgroot commented Jun 9, 2026

Copy link
Copy Markdown
Member

Users must be aware that they need to review their cookie consent
manager policies to make sure the botstopper cookie is going to be
allowed.

@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

.admonition-title {
background: transparent;
font-size: 16px;
margin: -12px -12px 6px;

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's nicer to set:

    padding: 0;
    margin-bottom: 6px;

that way we won't have conflicting padding and margins for the same element, above snippet results in the same front-end.

tdgroot added 2 commits June 9, 2026 15:58
Users must be aware that they need to review their cookie consent
manager policies to make sure the botstopper cookie is going to be
allowed.
This makes it take less space and more readable
@tdgroot tdgroot force-pushed the botstopper_cookie branch from aa43cb6 to 6f329f4 Compare June 9, 2026 13:59
@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

@tdgroot

tdgroot commented Jun 11, 2026

Copy link
Copy Markdown
Member Author

Closing, we made a different solution that requires no custom settings with cookie consent managers

@tdgroot tdgroot closed this Jun 11, 2026
@tdgroot tdgroot deployed to acceptance June 11, 2026 13:17 — with GitHub Actions Active
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants