WordPress.org

1. Justin_K
   Member
   Posted 2 years ago #
   

   Hey Mike,

   Love the plugin. Just one little thing: it would be nice not to have an "enforced" <small> tag around the CAPTCHA label; it breaks convention in my comment form, and the only way to get rid of it is to manually edit the code (si-captcha.php lines 234 & 236 on v2.6.5)

2. Mike Challis
   Member
   Plugin Author
   Posted 2 years ago #
   

   The next version will not have that, I just removed it

   Mike

3. Justin_K
   Member
   Posted 2 years ago #
   

   Sweet :)

   Just an update: After posting the above, I spent a bit more time playing with the plugin to make its layout "mesh" with my theme (Suffusion, one of the current top themes). It required one additional manual edit that you might consider making options:

   First, suffusion uses the full text "(required") rather than an asterisk to indicate a required field, and as it uses a "label left" layout, the "(required) text comes AFTER the input field (rather than after the label). You can see any post on my site for an example (www.justin-klein.com). Thus, I had to edit the code to remove <span class="required">*</span> from $label_string, and add <span class="required">(required)</span> after $input_string.

   You might consider making this a 3rd string which can be positioned either after $input_string or $label_string, enabling "edit-free" meshing with Suffusion or other themes which might mark "required" fields differently :)

4. Mike Challis
   Member
   Plugin Author
   Posted 2 years ago #
   

   Thanks for the suggestions.

   v2.6.5 does have a setting to adjust for a label layout.
   "CAPTCHA input label position on the comment form:"
   on suffusion, set it to "right".

   I just added a new setting in the "Text Labels:" to allow you to change the required field indicator.
   The default is "*", but you can now change it to "(required)" or anything you want.
   On suffusion you need to also change the label "CAPTCHA Code" to "CAPTCHA" to shorten it a bit.

   Now you should be able to set settings for suffusion all from the settings. I just loaded up suffusion theme and tested it.

   If you want to test it for me, download a fresh plugin zip file and replace the si-captcha-admin.php and si-captcha.php into the
   /wp-content/plugins/si-captcha-for-wordpress/ folder
   http://downloads.wordpress.org/plugin/si-captcha-for-wordpress.zip

   Mike

5. Justin_K
   Member
   Posted 2 years ago #
   

   Hey Mike,

   On Suffusion the label layout actually needs to be "Left" or it won't render correctly on IE7. "Left" will position the label and input box correctly, but the problem is that the "required" text will then appear *between* the label and the input box. The order (in the html) needs to be label,input,span-required - editing the plugin source to provide that order was the only way I could get it to look right in every browser, IE7 included.

6. Mike Challis
   Member
   Plugin Author
   Posted 2 years ago #
   

   Justin_K, Thanks for helping me.
   I added more label position options for the setting
   "CAPTCHA input label position on the comment form:"
   For suffusion theme you would use 'label-input-required'.

   If you want to test it for me, download a fresh plugin zip file and replace the si-captcha-admin.php and si-captcha.php into the
   /wp-content/plugins/si-captcha-for-wordpress/ folder
   http://downloads.wordpress.org/plugin/si-captcha-for-wordpress.zip
   Mike

7. Justin_K
   Member
   Posted 2 years ago #
   

   Perfect! Well done, I can finally kiss my manual code edits goodbye :)

Topic Closed

This topic has been closed to new replies.