Closed (fixed)
Project:
Honeypot
Version:
6.x-1.x-dev
Component:
User interface
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
14 May 2012 at 16:06 UTC
Updated:
1 Oct 2012 at 03:41 UTC
Jump to comment: Most recent file
Comments
Comment #1
geerlingguy commentedProbably has to do with the way I'm attaching the CSS. I might need to attach the CSS via an accessory function (in an #after_build function) rather than using drupal_add_css() inside honeypot_add_form_protection().
Won't have time to look into this for a few days, probably a couple weeks, but hopefully I'll get some time soon (otherwise, if someone else wants to take a crack at it, be my guest :).
Comment #2
jarchowk commentedI don't a lot of time either at the moment. Is this something I can put a band-aid on by hiding in my general css files, or would that be caught by spam bots?
Comment #3
geerlingguy commentedIf you move that CSS rule to your general CSS, it should help in the interim, and shouldn't make spam bots any different.
Comment #4
patrick r. commentedYou need to make use of the #attached key instead of drupal_add_css.
Comment #5
geerlingguy commentedAh, yes, forgot about #attached in Drupal 7 (though, even with #attached, I've sometimes found the file goes away for some weird reason). Anyways, will check out this patch soon and RTBC. I really need to take a few hours and commit some of these patches, but it'll probably have to wait until a Saturday.
Comment #6
geerlingguy commentedWorks great! I'm going to commit the patch in a few minutes.
Comment #7
geerlingguy commentedCommitted: http://drupalcode.org/project/honeypot.git/commit/89e152e
Comment #8
geerlingguy commentedBumping to 6.x. Attached patch should help in 6.x (using a
#pre_rendercallback in D6, since there's no#attachedproperty for forms like in D7).Comment #10
geerlingguy commentedComment #11
geerlingguy commented#8: 1578110-8-d6-form-css.patch queued for re-testing.
Comment #12
geerlingguy commentedReviewed patch, and it works perfectly, even with validation failures.
Comment #13
geerlingguy commentedFixed in 6.x-1.x-dev: http://drupalcode.org/project/honeypot.git/commit/c490002