Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
I see that the form is creating a lot of empty space by using "visibility: hidden" instead of "display: none". Is this an attempt to fool the spambots? It wasn't an issue before with version 1.3 (what I had installed before). I've checked Firebug with the page and my theme does not appear to be causing any conflicts.
Comment | File | Size | Author |
---|---|---|---|
#4 | spamicide-1.8-fix.patch | 21.17 KB | lirazsiri |
Comments
Comment #1
meustrus CreditAttribution: meustrus commentedAlso, was a permission removed for viewing the "Add spamicide to this form" link? I was using that to disable it in my admin (not user/1) account because the links are getting in the way.
Comment #2
lirazsiri CreditAttribution: lirazsiri commentedI'd like to confirm this is an issue.
Comment #3
meustrus CreditAttribution: meustrus commentedYes, it is an issue and still exists. I can't add Spamicide to forms in small places because the empty space is larger than the form itself (such as the Search form, to avoid DoS or just to get rid of the 'add spamicide to this form' link).
Comment #4
lirazsiri CreditAttribution: lirazsiri commentedTry this patch.
Comment #5
meustrus CreditAttribution: meustrus commentedYes, that worked. Thank you. There was a lot extra in that patch that I didn't apply, though, including:
if ($spamicide->enabled) {
spamicide.module.orig
The parts I did apply, though, I think are a fair improvement over the methods that I saw being replaced. Is this being committed?
Comment #6
lipcpro CreditAttribution: lipcpro commentedThanks lirazsiri for the patch it solved the problem of adding a class instead of using an id. I've been really frustrated by CVS, so now that d.o. is using Git I can be more responsive to my issue queue.
Thanks for your patience all