issues with captcha administration links [#169853]

Currently the CAPTCHA administration links on forms 'enable a captcha here', 'edit captcha type' link to the general CAPTCHA settings page with additional highlighting (by using the CSS class 'error') of the selection box for the relevant form_id. This design has the following problems:

It depends on the used theme. If the theme does not define a layout for the CSS class 'error', the relevant form_id will not be highlighted and the administrator will not see a difference between a normal CAPTCHA settings page, which makes it difficult to set/edit the captcha type for the form_id.

Using the 'error' class just for highlighting is bad style because there is no error.
The design might wrongly suggest to the administrator that all the presented settings (e.g. persistence, CAPTCHA description) are specifically for that form_id only, while these are general settings.

I'm responsible for this design but I was never really happy with it. Now I worked on an alternative which uses a separate page with just a selection box for choosing the CAPTCHA type for the form. I also added a link for the general settings page to the CAPTCHA administration links, which imitates the old behavior.
See attached patch.

Comment	File	Size	Author
#4	captcha_admin_links_4.patch	6.99 KB	soxofaan
#2	captcha_admin_links_3.patch	7.04 KB	soxofaan
#1	captcha_admin_links_2.patch	6.23 KB	soxofaan
	captcha_admin_links.patch	6.28 KB	soxofaan

Comments

Comment #1

soxofaan commented 24 August 2007 at 09:50

Status	File	Size
new	captcha_admin_links_2.patch	6.23 KB

updated patch against captcha.module version 1.42.2.24

BTW in my original post I hit submit instead of preview and Murphies Law stroke ;). The list should have been:

This design has the following problems:

It depends on the used theme. If the theme does not define a layout for the CSS class 'error', the relevant form_id will not be highlighted and the administrator will not see a difference between a normal CAPTCHA settings page, which makes it difficult to set/edit the captcha type for the form_id.

Using the 'error' class just for highlighting is bad style because there is no error.

The design might wrongly suggest to the administrator that all the presented settings (e.g. persistence, CAPTCHA description) are specifically for that form_id only, while these are general settings.