Skip to content
This repository has been archived by the owner on Jul 29, 2022. It is now read-only.

HTML5 form plugin, validation constraints & WAI-ARIA/accessibility #87

Closed
wants to merge 8 commits into from

Conversation

nfreear
Copy link

@nfreear nfreear commented Mar 14, 2013

Hi Adell,

Please pull (see bug #84):

  • Added a Javascript plugin: contributed/jquery.jeditable.html5.js
  • Added tests: tests/index.html
  • Added a demo page: html5.html

Additional supported types (so far):

  • "html5_text" (<input type=text >), "number", "email", "url".

Additional configuration properties:

  • inputmode, list, maxlength, pattern, required, min, max, step, html5_placeholder, html5_error_text.

Additional function:

  • $(selector).editableAriaShim()

HTML5 forms shim:

  • $(selector).checkValidity()

Many thanks, Nick

…for Jeditable

* It adds HTML5 form input types - eg. type="number"..
* ..Form validation constraints - eg. required, pattern= ..
* And, improves keyboard accessibility using WAI-ARIA roles etc.
…QUnit tests

* Needed to bump the jQuery version from 1.2.6 to 1.4.4 (likely minimum)
…tests

* Fix robustness of `.checkValidity()`
* Add function `typeof` unit tests to `tests/index.html`
…nternet Explorer

* _supportInType private function based on "Dive into HTML5" example,
* http://diveintohtml5.info/everything.html
* Re-order QUnit tests
@nfreear
Copy link
Author

nfreear commented Mar 21, 2013

Hi Adell,
As you can see, I've added some more useful commits. I've finished work for now - so its a suitable time to pull - up to nfreear/jquery_jeditable@838e2ecad45faa54.

Yours, Nick

NicolasCARPi added a commit that referenced this pull request Feb 20, 2018
So I almost added all of the code from the pull. Fixed the tab key
triggering edit, now you can tab the inputs and if you type another key
it'll go into edit mode. Instead of adding a plugin, I added it in the
code directly, and merged the text type with html5_text because now
everyone should do html5 anyway :)
@NicolasCARPi
Copy link
Owner

Hello @nfreear,

Thank you very much for this very good pull request :)

I have merged most of your changes!

Cheers,
~Nico

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants