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.

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

Placeholder text is actually submitted with the form #5

Open
brendon opened this issue Jun 26, 2012 · 2 comments
Open

Placeholder text is actually submitted with the form #5

brendon opened this issue Jun 26, 2012 · 2 comments

Comments

@brendon
Copy link

brendon commented Jun 26, 2012

Hi there, thanks again for this cool javascript :)

I've found a problem where the placeholder text is actually submitted with the form in IE. This leads to validations not being triggered (e.g. validating the presence of a string). Ideally the form placeholders should be emptied out prior to submission if they have that secret character in them as this is how the placeholder works in other browsers. Is the current behavior by design or is it a bug? :)

@ghost ghost assigned amiel Jun 28, 2012
@amiel
Copy link
Owner

amiel commented Jun 28, 2012

Hi @brendon,

This behavior is sort of by design (I'm trying to make this the simplest thing that could possibly work). But I think it might be time for html5support to handle this.

So, I'll happily merge a pull request, otherwise I'll probably get to it next month.

@brendon
Copy link
Author

brendon commented Jul 4, 2012

Hi Amiel, that's all good. In our situation the field was optional so this
had undesired effects when it wasn't filled out. I ended up having to use
another plugin.

Thanks heaps for getting back to me. :)

Have a great day!

Brendon Muir

On Fri, Jun 29, 2012 at 6:25 AM, Amiel Martin <
[email protected]

wrote:

Hi @brendon,

This behavior is sort of by design (I'm trying to make this the simplest
thing that could possibly work). But I think it might be time for
html5support to handle this.

So, I'll happily merge a pull request, otherwise I'll probably get to it
next month.


Reply to this email directly or view it on GitHub:
#5 (comment)

@amiel amiel removed their assignment Mar 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants