I had a few problems with the audio captcha in IE6, IE7, and Safari. In most cases the player would appear, but I couldn't get the audio to play. Firefox and Opera worked pretty well. I made a few changes to get it to work in those browsers, but Opera seems to have issues now. For the project I'm working on, not having an audio captcha in Opera is acceptable so I'm just going to pass along the changes I've made so far.

The patch also includes the addition of a maxlength property on the captcha text field and an anchor on the audio captcha link so it jumps back to the link rather than the top of the page (better for screen readers, but just as annoying to anyone not using one).

By the way, excellent job on this module. Captchas tend to be accessibility killers, but this one works quite well.

CommentFileSizeAuthor
mollom.css_.txt381 bytesjcarnett
mollom.module-5.x-1.1.patch3.97 KBjcarnett

Comments

dries’s picture

dries commented

jcarnett, thanks for the patches -- that is helpful. I'm a bit reluctant to commit the CSS patch. The module changes look OK, but the CSS patch has a lot of browser-specific hacks that I'd like to avoid, really. Sure there isn't a cleaner way to do this?

jcarnett’s picture

jcarnett commented

CSS is definitely not my strength. I'll send it over to a designer I know and get him to double-check it for me.

jcarnett’s picture

jcarnett commented

Alright I had some extra eyes take a look at it. To give a little background, the double object tag approach is based on this A List Apart article: http://www.alistapart.com/articles/byebyeembed/. See the "Embedding QuickTime movies for everyone else" section. They go on to use IE conditional comments to hide the second tag, but having to use a syntax hack there seemed ugly to me so I used CSS to hide the second object in IE. Of course the CSS still uses hacks, so maybe it's no better. That CSS is based on another article talking about the same thing, but I can't seem to find that one again. At any rate, it seems that using this double-object approach to do a valid embed requires one or the other. The ideal solution might be to use Flash.

dries’s picture

dries commented
Status: Needs review » Fixed

This was fixed in Mollom 1.6 for both DRUPAL-5 and DRUPAL-6. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.

aequor’s picture

aequor commented

i reset to defaults and given new public, private values at mollom keys form, for me it worked fine... thanx for above all suggestion which helps me alot...

----
RamMohan