mirror of
https://github.com/airbnb/javascript.git
synced 2026-04-25 03:00:19 -04:00
[jQuery] better performance for DOM queries. fixes #22
This commit is contained in:
23
README.md
23
README.md
@@ -1178,20 +1178,27 @@
|
||||
}
|
||||
```
|
||||
|
||||
- Scope jQuery object queries with find. [jsPerf](http://jsperf.com/jquery-find-vs-context-sel/13)
|
||||
- For DOM queries use Cascading `$('.sidebar ul')` or parent > child `$('.sidebar > .ul')`. [jsPerf](http://jsperf.com/jquery-find-vs-context-sel/16)
|
||||
- Use `find` with scoped jQuery object queries.
|
||||
|
||||
```javascript
|
||||
// bad
|
||||
$('.sidebar ul').hide();
|
||||
|
||||
// bad
|
||||
$('.sidebar > ul').hide();
|
||||
|
||||
// bad
|
||||
$('.sidebar', 'ul').hide();
|
||||
|
||||
// good
|
||||
// bad
|
||||
$('.sidebar').find('ul').hide();
|
||||
|
||||
// good
|
||||
$('.sidebar ul').hide();
|
||||
|
||||
// good
|
||||
$('.sidebar > ul').hide();
|
||||
|
||||
// good
|
||||
$sidebar.find('ul');
|
||||
|
||||
// good
|
||||
$($sidebar[0]).find('ul');
|
||||
```
|
||||
|
||||
**[[⬆]](#TOC)**
|
||||
|
||||
Reference in New Issue
Block a user