search

5afe / safe-landing-page

Deprecated, new repo – https://github.com/safe-global/safe-homepage
MIT License
3 stars 7 forks source link

Update cookie page to include Beamer cookies #17

Closed germartinez closed 2 years ago

germartinez commented 2 years ago

What it solves

Resolves #14

How this PR fixes it

Includes the 3rd party cookies added by Beamer as present in https://www.getbeamer.com/docs#cookies

How to test it

Verify that the "Cookie Policy" page display the rightful information as per Beamer official cookie documentation.

Screenshots

Screen Shot 2022-02-18 at 11 53 35

github-actions[bot] commented 2 years ago

CLA Assistant Lite All Contributors have signed the CLA.

germartinez commented 2 years ago

recheckcla

germartinez commented 2 years ago

@DiogoSoaress I think it now makes sense to increase the width of Td1 component to something like 200-250px. What do you think?

DiogoSoaress commented 2 years ago

@DiogoSoaress I think it now makes sense to increase the width of Td1 component to something like 200-250px. What do you thing?

I think 200px looks great! Good suggestion as these cookies ids are longer than the already existent ones. Screen Shot 2022-02-18 at 15 45 36