Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

Some code uses $.isArray, some uses Array.isArray #1096

Closed
rwaldron opened this issue Jun 21, 2012 · 3 comments
Closed

Some code uses $.isArray, some uses Array.isArray #1096

rwaldron opened this issue Jun 21, 2012 · 3 comments
Assignees
@jasonsanjose
Labels
code cleanup

Comments

@rwaldron
Copy link

If this is meant to be used with modern browsers only, then I was expect Array.isArray to preferred.
@njx
Copy link

njx commented Jun 21, 2012

Good catch, thanks.

@ghost ghost assigned jasonsanjose Jun 21, 2012
@jasonsanjose
Copy link
Member

Nominating sprint 11

@pthiess
Copy link
Contributor

pthiess commented Jun 22, 2012

reviewed - @jason-sanjose

@jasonsanjose jasonsanjose mentioned this issue Jun 28, 2012
Merged
@rwaldron rwaldron mentioned this issue Jul 8, 2012
Closed
@peterflynn peterflynn mentioned this issue Jul 11, 2012
Merged
@DennisKehrig DennisKehrig mentioned this issue Jul 12, 2012
Merged
@core-ai-bot core-ai-bot mentioned this issue Aug 29, 2021
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@jasonsanjose jasonsanjose

Labels
code cleanup
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@rwaldron @njx @jasonsanjose @pthiess