Skip to content
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

Remove deceptive "Accept all" button colors/contrast #8

Closed
geek-andi opened this issue Jun 28, 2021 · 7 comments
Closed

Remove deceptive "Accept all" button colors/contrast #8

geek-andi opened this issue Jun 28, 2021 · 7 comments

Comments

@geek-andi
Copy link

Problem

By law, users must be given a clear yes/no option. Max Schrems claims that deceptive colors/contrast does not comply to GDPR, I think he's right :) (even if courts end up judging differently, we should not "nudge" users to the one or the other option). Source: https://noyb.eu/en/noyb-aims-end-cookie-banner-terror-and-issues-more-500-gdpr-complaints

Currently, cookiesjsr does exactly this.

image

image

  • The "Accept all" button is highlighted with another color fill/border than the other buttons
  • The video blocking info has "Accept all cookies" in all-caps, bigger font size and highlighted bg color

Proposed solution

Remove highlighting the "Accept all" buttons in the default styling. (To make the banner visually easier to get, the cookie setting button could be smaller.)

(Of course users can override the css, but I think the default settings should be sensible)

@geek-andi geek-andi changed the title Remove deceptive button colors/contrast Remove deceptive "Accept all" button colors/contrast Jun 28, 2021
@jfeltkamp
Copy link
Owner

I'm absolute okay with your interest, but the buttons have the same contrast, in so far as white/blue has the same contrast-value as blue/white. The buttons have the same size and font-size. "Deceptive" means that buttons appear as "disabled", "disadvantageous" or "dangerous" or something like this. You can't even say that either of these buttons is clearly highlighted. Which of the two would that be? The one with the border, or the monochrome one.

But I will leave this issue open to catch other comments and meanings on it.

@geek-andi
Copy link
Author

@jfeltkamp Thanks for your answer! Yes, there is some room for discussion here.

For me, two things are quite obvious:

  1. In the first screenshot bigger font size and capital letters are used the "Accept all cookies" button. "Only accept Video cookies" is not even a button
  2. In the second screenshot the "Accept all" Button has a color fill which makes it more visible to me. Since there are two buttons that have a white fill color, the one button which is different will always stand out

Indeed, I'm not sure if deceptive is the correct word for it. At least I would say both designs nudge users to click on the button which is less privacy friendly. In my opinion that should not be the case. Because when you think about the dozens of banners average users "click away" each days it affects the choice users make.

@JPustkuchen
Copy link

Please see https://www.drupal.org/project/cookies/issues/3250618 for similar issue on drupal.org and suggestions for a flexible solution by @thomas.frobieter

@JPustkuchen
Copy link

JPustkuchen commented Mar 2, 2022

@jfeltkamp could we please at least add semantic classes to the buttons, so we can safely style them? This should be no breaking change.

Current situation is the following:
Bar:
grafik
Modal layer:
grafik

So I'd suggest to simply add

  • "acceptAll" (like already existing denyAll) - in both dialogs
  • "save" (or cookiejsr-settings-save???) - in modal layer

In a later cleanup (future task), the BEM style should be reviewed & revised and I'd vote to remove the "important" class.

@jfeltkamp
Copy link
Owner

Ok, I improved that in the master of jfeltkamp/cookiesjsr.
Please check, if it works for you. Then I will tag it and deploy on CDN.

To test with Drupal:
Go to libraries and checkout cookiesjsr as master.
(correct version, when you see 1.0.13 in the pacckage.json)
Un-check "Load by CDN" in COOKiES settings.

In the banner:
Allow and Deny get both "important" class and an additional individual class.
Responsive behavior doesen't display-none the deny button anymore and alternativly swap to mobil mode earlier, if there is not enough place for all buttons in a row.

In the layer:
The "Save" action got the important class. Allow and Deny don't have "important" class anymore, and will also be handled equal in responsive.

@JPustkuchen
Copy link

Thank you @jfeltkamp! Green lights from our side :)

@geek-andi
Copy link
Author

geek-andi commented Dec 7, 2022

Nice, this seems to be fixed. Thanks @jfeltkamp !

See https://www.drupal.org/project/cookies/issues/3250618

grafik

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants