mirror of
https://github.com/Mastermindzh/react-cookie-consent.git
synced 2025-01-20 18:41:44 +01:00
Update README.md
Added an explanation of the legacy cookie to the readme.
This commit is contained in:
parent
13c6909b1a
commit
a56e806ee2
18
README.md
18
README.md
@ -22,6 +22,7 @@ Example branch: https://github.com/Mastermindzh/react-cookie-consent/tree/exampl
|
||||
- [Using it](#using-it)
|
||||
- [Props](#props)
|
||||
- [Debugging it](#debugging-it)
|
||||
- [Why are there two cookies? One of which named "Legacy"](#why-are-there-two-cookies--one-of-which-named--legacy-)
|
||||
- [Styling it](#styling-it)
|
||||
- [Examples](#examples)
|
||||
- [changing the bar background to red](#changing-the-bar-background-to-red)
|
||||
@ -163,6 +164,23 @@ Because the cookie consent bar will be hidden once accepted, you will have to se
|
||||
|
||||
**Note:** Don't forget to remove the `debug`-property for production.
|
||||
|
||||
## Why are there two cookies? One of which named "Legacy"
|
||||
|
||||
The short story is that some browsers don't support the SameSite=None attribute.
|
||||
The modern browsers force you to have SameSite set to something other than none.
|
||||
|
||||
So react-cookie-consent fixes this like so:
|
||||
|
||||
- set the fallback cookie (e.g -legacy) first, this will always succeed (on all browsers)
|
||||
- set the correct cookie second (this will work on modern browsers, fail on older ones)
|
||||
|
||||
This happens on lines [186-192](https://github.com/Mastermindzh/react-cookie-consent/blob/master/src/index.js#L186-L192)
|
||||
|
||||
When checking the cookie it'll do it in reverse. If the regular cookie exists, it'll use that. If no regular cookie exists it'll check whether the legacy cookie exists. If both are non-existent no consent was given.
|
||||
|
||||
The long story can be found here: [pull-request#68](https://github.com/Mastermindzh/react-cookie-consent/pull/68)
|
||||
|
||||
|
||||
## Styling it
|
||||
|
||||
You can provide styling for the bar, the button and the content. Note that the bar has a `display: flex` property as default and is parent to its children "content" and "button".
|
||||
|
Loading…
Reference in New Issue
Block a user