Problem/Motivation

Currently the "opt in with categories" method is incomplete for the Italian Cookie Law because the "reject all" cookie button is not available.

The Italian cookie law requires the following buttons:

  • Allow all
  • Deny all (or X / Close button to deny all)
  • Manage preferences

Currently the module opt-ins method are two:

  1. opt-in: display "allow all" and "deny all"
  2. opt-in with categories: display "allow all" and "manage preferences"

There isn't an option with all three buttons.

I've created a simple patch to solve the problem.

CommentFileSizeAuthor
#4 eu_cookie_compliance-enable-deny-on-categories-3263244-4.patch3.08 KBfinex
#2 eu_cookie_compliance-enable-deny-on-categories-3263244.patch3.08 KBfinex

Issue fork eu_cookie_compliance-3263244

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

FiNeX created an issue. See original summary.

finex’s picture

finex commented
StatusFileSize
new eu_cookie_compliance-enable-deny-on-categories-3263244.patch3.08 KB

Patch to add "Deny all" button on "opt-in with categories" method as required by Italian cookie law.

finex’s picture

finex commented
Issue summary: View changes
finex’s picture

finex commented
StatusFileSize
new eu_cookie_compliance-enable-deny-on-categories-3263244-4.patch3.08 KB

Updated patch.

svenryen’s picture

svenryen commented

Thanks for the patch. I'll try to review it this weekend.

pietrocap’s picture

pietrocap commented

Hi,
I need this patch but for Drupal 7. Could You gently help me?

svenryen’s picture

svenryen commented

Hi @pietrocap, I stopped backporting to Drupal 7, but maybe the community can help?

svenryen’s picture

svenryen commented
Status: Needs review » Needs work

@FiNeX, your patch needs more work.

Currently, the patch introduces a third button for all sites that show the banner when they use Opt-in with categories. Some site owners may not agree with this sudden change. Can you please change your patch so that the new feature is optional? A checkbox to decide whether to use the feature should be sufficient.

svenryen’s picture

svenryen commented
Category: Bug report » Feature request
propercroc’s picture

propercroc commented

I like this. My german client also demands we have a deny-all-button. Included the patch in composer and added some styles, voilá!
Thanks!

svenryen’s picture

svenryen commented

Bump the timestamp of this issue so that we don't lose track of it for -beta4.

svenryen credited zanonmark.

svenryen’s picture

svenryen commented

svenryen opened merge request !90

svenryen’s picture

svenryen commented
Status: Needs work » Fixed

  • svenryen committed e54334a on 8.x-1.x 
    Issue #3263244 by svenryen, FiNeX, zanonmark: Enable "deny all" button...

  • svenryen committed be410ab on 8.x-1.x 
    Issue #3263244 by FiNeX, svenryen, pietrocap, ProperCroc, zanonmark:...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.