Skip to content
This repository has been archived by the owner on Mar 15, 2018. It is now read-only.

Fix recaptcha on AMO (bugs 589359, 932514) #1410

Merged
merged 1 commit into from Nov 20, 2013

Conversation

mattbasta
Copy link
Contributor

This fix manages to not change the entire login implementation to fix what otherwise happens to be an unending series of terrifically simple bugs.

  • Bug 932514: Can't switch back to image from audio
  • Bug 589359: Seriously why is this still a thing

cc @cvan, @ngokevin

@@ -13,7 +13,7 @@
<p>
If this is hard to read, you can
<a href="#" id="recaptcha_different">try different words</a> or
<a href="#" id="recaptcha_audio">listen to something</a> instead.
<a href="#" id="recaptcha_audio">try a different type of challenge</a> instead.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is #recaptcha_audio the correct id then?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not the best ID, but it's fine.

@ngokevin
Copy link
Contributor

r+

mattbasta added a commit that referenced this pull request Nov 20, 2013
Fix recaptcha on AMO (bugs 589359, 932514)
@mattbasta mattbasta merged commit 4c0f115 into mozilla:master Nov 20, 2013
@mattbasta mattbasta deleted the recaptchafix branch November 20, 2013 00:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants