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

[#1765] test for classList.forEach before using #1805

Merged
merged 1 commit into from
Sep 17, 2015

Conversation

gabemart
Copy link
Contributor

Safari throws an error when using the new elHasClass helper function.

Test if classList.forEach is defined - if not, default back to using the regex to test for class presence.
#1765

@akvobot
Copy link

akvobot commented Sep 16, 2015

Checklist

  • Test plan
  • Copyright header
  • Code formatting
  • Documentation

@gabemart gabemart assigned gabemart and KasperBrandt and unassigned gabemart Sep 16, 2015
@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling 963cb6f on akvo:issue/1765-remove-zepto into 53ff887 on akvo:develop.

KasperBrandt added a commit that referenced this pull request Sep 17, 2015
[#1765] test for classList.forEach before using
@KasperBrandt KasperBrandt merged commit 3284816 into develop Sep 17, 2015
@KasperBrandt KasperBrandt deleted the issue/1765-remove-zepto branch September 17, 2015 07:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

5 participants