Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Plugin Does Not Always Work In IE7/8 #26

Closed
alexbeardsley opened this issue Feb 17, 2015 · 2 comments
Closed

Plugin Does Not Always Work In IE7/8 #26

alexbeardsley opened this issue Feb 17, 2015 · 2 comments
Assignees
@iamthechad
Labels
minor
Milestone
Version 1.1

Comments

@alexbeardsley
Copy link
Contributor

The plugin outputs the script tag immediately in-line where it is present in the GSP. In IE7 and 8, which reCaptca is supposed to be compatible with, script tags aren't executed when inside a div. One workaround is to add the option to not output the script tag via the recaptcha{..} config block, and instead the developer can put the script tag elsewhere.
@iamthechad iamthechad self-assigned this Feb 17, 2015
@alexbeardsley
Copy link
Contributor Author

Created a pull request: #27

@iamthechad iamthechad added this to the Version 1.1 milestone Feb 17, 2015
@iamthechad
Copy link
Owner

Opened new PR #28 to properly target the correct branch.

iamthechad added a commit that referenced this issue Feb 18, 2015
@iamthechad
More changes to allow separation of script tag
d8a3548 
Allow the script tag to be placed elsewhere than the captcha HTML. Refs #26
iamthechad added a commit that referenced this issue Feb 18, 2015
@iamthechad
Update doc
0b2e176 
Refs #26
iamthechad added a commit that referenced this issue Feb 19, 2015
@iamthechad
Properly check validation response
896d94a 
Restore correct checking of JSON response from ReCaptcha server. Plugin will now work again. Refs #26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@iamthechad iamthechad

Labels
minor
Projects
None yet
Milestone
Version 1.1
Development

No branches or pull requests
2 participants
@iamthechad @alexbeardsley