enabling Add CAPTCHA administration links to forms, adds captcha to all forms even custom search blocks for logged in user.

I understand that this is by design, but without the fieldset classed out, themers cannot hide this when it appears in areas of a theme it shouldn't. For example, any custom search block added to the header area of a theme, will automatically get this captcha option.. and to disable it, a themer must hide all fieldsets in that region.. if there were a class added to the fieldset, we could easily just hide that captcha fieldsets in whatever areas are undesirable.

I have attached an image which should illustrate the problem

Files: 
CommentFileSizeAuthor
#3 captcha-admin-links.png74.35 KBKrisBulman
#2 1142606_captchaadminlinksclass.patch557 bytessoxofaan
PASSED: [[SimpleTest]]: [MySQL] 733 pass(es). View
captchaclass.jpg120.53 KBKrisBulman

Comments

KrisBulman’s picture

Title: enabling Add CAPTCHA administration links to forms, adds captcha to all forms even custom search blocks » add a captcha class to the wrapping fieldset for 'place a captcha here'
soxofaan’s picture

Title: add a captcha class to the wrapping fieldset for 'place a captcha here' » add a css class to the wrapping fieldset for 'place a captcha here'
Version: 6.x-2.4 » 6.x-2.x-dev
Issue tags: +low-hanging fruit
FileSize
557 bytes
PASSED: [[SimpleTest]]: [MySQL] 733 pass(es). View

I haven't got the time to try it out, but the following should do the trick I suppose

KrisBulman’s picture

Status: Active » Reviewed & tested by the community
FileSize
74.35 KB
git status
# On branch 6.x-2.x
git apply -v 1142606_captchaadminlinksclass.patch Checking patch captcha.module...
Applied patch captcha.module cleanly.

captcha-admin-links class added to fieldset properly

soxofaan’s picture

Status: Reviewed & tested by the community » Closed (fixed)