Closed (fixed)
Project:
Text CAPTCHA
Version:
7.x-1.x-dev
Component:
Code
Priority:
Major
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
7 Jun 2013 at 17:19 UTC
Updated:
23 Sep 2013 at 19:01 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
bryanhirsch commentedProposed patch attached. This does two things:
1. prepends Challenge Description in front of text-based challenge
2. form_alters forms with text captcha challenges to unset the duplicative fieldset description provided by captcha module
Comment #2
Anonymous (not verified) commentedRe-applied to 7.x-1.x
Comment #3
bryanhirsch commentedAfter testing a bit, I think this patch is a better approach. Here's what does:
1. Create a new text field on the admin page for a Text Captcha challenge description
2. Prepends this before the text captcha challenge question
3. Wraps challenge question in span tags so it can be themed to be visually distinct from the description
Screen shot here.
Unlike the first patch I submitted, this does not unset Captcha's Challenge Description. In retrospect, that approach was too aggressive.
Comment #4
theneonlobster commentedI have made a couple minor adjustments to the patch, replacing the span with a div, and wrapping the Text Captcha challenge description in a span for easier styling.
Comment #5
theneonlobster commentedThe previous patch would not apply to the 7.x-1.x branch. The attached patch will.
Comment #6
bryanhirsch commented@theneonlobster, thanks. Good improvements.
Comment #7
bryanhirsch commented