Skip to content

Keyboard polyfill #61

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

Closed
wants to merge 2 commits into from
Closed

Keyboard polyfill #61

wants to merge 2 commits into from

Conversation

l15k4
Copy link
Contributor

@l15k4 l15k4 commented Oct 29, 2014

Based on this conversation I'd like to discuss how to include this PR into scala-js-dom... It starts to be a little messy in there and we should think about a system that would allow for adding new extensions, polyfills and other cool things more easily... Meaning that it would be clear how and where to add it.

For instance here the object KeyboardPolyfill is a polyfill thingy, the KCode is a keyboard thingy and ChCode rather Character thingy :-) Anyway all this stuff is quite essential if you want to do something really keyboard interactive...

The polyfill's normalize method will probably grow as people will notice differences on different devices/browsers, which is mainly mobile devices and ios.

@l15k4 l15k4 force-pushed the keyboard-polyfill branch from d0a899e to d270403 Compare October 29, 2014 23:11
@l15k4 l15k4 force-pushed the keyboard-polyfill branch from d270403 to 5589495 Compare October 29, 2014 23:13
@Voltir
Copy link

Voltir commented Jan 16, 2015

I would use this - I am using this, but I just duplicated the file and am using it locally.

@l15k4
Copy link
Contributor Author

l15k4 commented Jan 17, 2015

I made it available as a library scalajs-keyboard-polyfil. Note it is not released or published yet until scalajs 6.0.0 is out.

@mseddon
Copy link
Contributor

mseddon commented Jan 20, 2016

@sjrd we can close this one- scala-js-dom does not implement polyfills.

@l15k4
Copy link
Contributor Author

l15k4 commented Jan 20, 2016

Sure, oss library is more suitable here...

@l15k4 l15k4 closed this Jan 20, 2016
Visbone pushed a commit to Visbone/scala-js-dom that referenced this pull request Mar 18, 2021
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

Successfully merging this pull request may close these issues.

3 participants