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

Browser support #12

Open
slavanga opened this issue Apr 24, 2016 · 4 comments
Open

Browser support #12

slavanga opened this issue Apr 24, 2016 · 4 comments
Assignees

Comments

@slavanga
Copy link

What browsers does Modaal actively support?

@DioNNiS
Copy link

DioNNiS commented Apr 24, 2016

+1

@mynamesleon
Copy link

mynamesleon commented Apr 25, 2016

+1 for having the info, but an increase in actual browser support would also be nice.

Given that this is supposedly a modal plugin for "furthering the cause for an accessible web", it's a shame it doesn't work in at least IE8; IE7 support would also be nice.

If the support is to remain IE9+, then it would be nice to see jQuery removed as a dependency. But if jQuery is to remain a dependency, it seems silly not to leverage jQuery v1's features fully to support older browsers and make this as accessible as possible.

(Obviously browser support isn't an explicit part of WCAG, but in my experience clients (and devs) that truly care about web accessibility typically care about optimal browser support as well)

@cbotman cbotman mentioned this issue Apr 26, 2016
@danhumaan danhumaan self-assigned this May 18, 2016
@danhumaan
Copy link
Collaborator

Thanks for raising this issue and my apologies for the delay in getting back to you.

I can confirm that Modaal is only supported in IE9+ at the moment due to the Object.create() method not being supported in IE8. We'll review what's necessary to increase the browser support back to ie8 and possibly 7, and update this task accordingly.

To quickly touch on the jQuery dependency, this is certainly something that's being considered and planned, as is seen in #14, so please refer to that ticket for further information.

@danb-humaan
Copy link
Contributor

Pencilling this in on the v0.4.0 milestone for us to compile a proper browser support chart. Would also be worth adding IE8 support.

@danb-humaan danb-humaan added this to the v0.4.0 milestone Mar 16, 2017
@danhumaan danhumaan removed this from the v0.4.0 milestone May 29, 2018
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

5 participants