Closed (won't fix)
Project:
Drupal.org security advisory coverage applications
Component:
module
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
9 Oct 2012 at 12:01 UTC
Updated:
12 Aug 2013 at 13:47 UTC
Jump to comment: Most recent file
Comments
Comment #1
Ignigena commentedMake sure to add a README.txt file to your module.
Also, noticed quite a few minor errors and warnings mostly related to Drupal Coding Standards. These are all easy fixes, but good things to check before submitting here. Here are the errors that were found using PAReview:
http://ventral.org/pareview/httpgitdrupalorgsandboxbassie1111806384git
Comment #2
sittard commentedNice idea and I'm sure many sites would benefit from this module. I like the concept that you can set your privacy settings for all sites utilising the Silk Tide Cookie Consent Plugin. I always felt it was a bit of a shame that the cookie law placed the responsibility on web site owners to comply rather than working with the browser developers to find a workable solution.
Some initial feedback:
Please include some instructions on how to install this module. It would also be nice to see some information on what this module does and how it works on the project screen. I would also suggest adding a paragraph explaining what cookies this modules sets.
On the silktide website you can test if Cookie conset is working by adding a query parameter.
I'm not sure if you have allowed for this or not, but some instructions on testing if Cookie Consent is working or not would be a bonus.
You have a overlay conflict with the Admin Menu module (http://drupal.org/project/admin_menu) which unfortunately means that the Cookie Consite Popup always hides the admin menu (until you click consent).
On a clean Drupal install I also got some error messages (see attached file).
I also notice that the popup did not activate in FireFox v16 but worked fine in IE9 and Chrome.
Comment #3
klausiClosing due to lack of activity. Feel free to reopen if you are still working on this application.
Comment #4
knalstaaf commentedIt would be very welcome to have this as a Drupal module.