Skip to content

Commit f00efa7

Browse files
fiegfieg
committed
Merge pull request #243 from fieg/sjorsrijsdam-issue-93
RE: Fix multiple instances bug
2 parents ada44e5 + f3b3ae2 commit f00efa7

File tree

7 files changed

+441
-6
lines changed

7 files changed

+441
-6
lines changed

‎.travis.yml‎

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -20,10 +20,9 @@ node_js:
2020
- 0.10
2121

2222
env:
23-
- JQUERY_VERSION=1.7.*
24-
- JQUERY_VERSION=1.8.*
25-
- JQUERY_VERSION=1.9.*
2623
- JQUERY_VERSION=1.10.*
2724
- JQUERY_VERSION=1.11.*
25+
- JQUERY_VERSION=1.12.*
2826
- JQUERY_VERSION=2.0.*
2927
- JQUERY_VERSION=2.1.*
28+
- JQUERY_VERSION=2.2.*

‎src/jquery-ias.js‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@
7878
* @returns {object}
7979
*/
8080
this.getItemsContainer = function() {
81-
return $(this.itemsContainerSelector);
81+
return $(this.itemsContainerSelector, this.$container);
8282
};
8383

8484
/**
@@ -245,7 +245,7 @@
245245
}
246246
});
247247
});
248-
248+
249249
promise.fail(function() {
250250
if (callback) {
251251
callback();

‎test/07-multiple-instances-test.js‎

Lines changed: 63 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,63 @@
1+
describe('IAS', function () {
2+
before(function () {
3+
var deferred = when.defer();
4+
5+
this.timeout = 10000;
6+
7+
loadFixture('multiple1.html', function () {
8+
[$('.overflow1'), $('.overflow2')].forEach(function ($overflow) {
9+
$overflow.ias({
10+
container: '.container',
11+
item: '.item',
12+
pagination: '.pagination',
13+
next: '.next',
14+
});
15+
16+
deferred.resolve();
17+
});
18+
});
19+
20+
return deferred.promise;
21+
});
22+
23+
after(function() {
24+
jQuery.ias('destroy');
25+
});
26+
27+
function runTest(scrollMoreSelector, scrollLessSelector) {
28+
var deferred = when.defer();
29+
30+
scrollDown(100, $(scrollMoreSelector)).then(function () {
31+
wait(1000).then(function () {
32+
expect($(scrollMoreSelector).find('.item').length).toEqual(20);
33+
expect($(scrollLessSelector).find('.item').length).toEqual(10);
34+
35+
scrollDown(100, $(scrollLessSelector)).then(function () {
36+
wait(1000).then(function () {
37+
expect($(scrollMoreSelector).find('.item').length).toEqual(20);
38+
expect($(scrollLessSelector).find('.item').length).toEqual(20);
39+
40+
scrollDown(100, $(scrollMoreSelector)).then(function () {
41+
wait(1000).then(function () {
42+
expect($(scrollMoreSelector).find('.item').length).toEqual(30);
43+
expect($(scrollLessSelector).find('.item').length).toEqual(20);
44+
45+
deferred.resolve();
46+
});
47+
});
48+
});
49+
});
50+
});
51+
});
52+
53+
return deferred.promise;
54+
}
55+
56+
it('should scroll the first more', function () {
57+
return runTest('.overflow1', '.overflow2');
58+
});
59+
60+
it('should scroll the second more', function () {
61+
return runTest('.overflow2', '.overflow1');
62+
});
63+
});

‎test/buster.js‎

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,10 @@ config["My tests"] = {
3232
{ path: "/framed.html", file: "test/fixtures/framed.html" },
3333
{ path: "/short.html", file: "test/fixtures/short.html" },
3434
{ path: "/ajax1.html", file: "test/fixtures/ajax1.html" },
35-
{ path: "/ajax2.html", file: "test/fixtures/ajax2.html" }
35+
{ path: "/ajax2.html", file: "test/fixtures/ajax2.html" },
36+
{ path: "/multiple1.html", file: "test/fixtures/multiple1.html" },
37+
{ path: "/multiple2.html", file: "test/fixtures/multiple2.html" },
38+
{ path: "/multiple3.html", file: "test/fixtures/multiple3.html" }
3639
],
3740
specs: [
3841
"test/*-test.js"

‎test/fixtures/multiple1.html‎

Lines changed: 124 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,124 @@
1+
<!DOCTYPE html>
2+
<html>
3+
<head>
4+
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
5+
<title>Framed</title>
6+
</head>
7+
<body>
8+
<div class="overflow1" style="width: 400px; height: 400px; overflow: scroll">
9+
<div class="container">
10+
<div class="item">
11+
<strong>item 1</strong>
12+
<p>Pellentesque habitant morbi tristique senectus et netus et malesuada fames ac turpis egestas. Vestibulum tortor quam, feugiat vitae, ultricies eget, tempor sit amet, ante. Donec eu libero sit amet quam egestas semper. Aenean ultricies mi vitae est. Mauris placerat eleifend leo. Quisque sit amet est et sapien ullamcorper pharetra. Vestibulum erat wisi, condimentum sed, commodo vitae, ornare sit amet, wisi. Aenean fermentum, elit eget tincidunt condimentum, eros ipsum rutrum orci, sagittis tempus lacus enim ac dui. Donec non enim in turpis pulvinar facilisis. Ut felis.</p>
13+
</div>
14+
15+
<div class="item">
16+
<strong>item 2</strong>
17+
<p>Pellentesque habitant morbi tristique senectus et netus et malesuada fames ac turpis egestas. Vestibulum tortor quam, feugiat vitae, ultricies eget, tempor sit amet, ante. Donec eu libero sit amet quam egestas semper. Aenean ultricies mi vitae est. Mauris placerat eleifend leo. Quisque sit amet est et sapien ullamcorper pharetra. Vestibulum erat wisi, condimentum sed, commodo vitae, ornare sit amet, wisi. Aenean fermentum, elit eget tincidunt condimentum, eros ipsum rutrum orci, sagittis tempus lacus enim ac dui. Donec non enim in turpis pulvinar facilisis. Ut felis.</p>
18+
</div>
19+
20+
<div class="item">
21+
<strong>item 3</strong>
22+
<p>Pellentesque habitant morbi tristique senectus et netus et malesuada fames ac turpis egestas. Vestibulum tortor quam, feugiat vitae, ultricies eget, tempor sit amet, ante. Donec eu libero sit amet quam egestas semper. Aenean ultricies mi vitae est. Mauris placerat eleifend leo. Quisque sit amet est et sapien ullamcorper pharetra. Vestibulum erat wisi, condimentum sed, commodo vitae, ornare sit amet, wisi. Aenean fermentum, elit eget tincidunt condimentum, eros ipsum rutrum orci, sagittis tempus lacus enim ac dui. Donec non enim in turpis pulvinar facilisis. Ut felis.</p>
23+
</div>
24+
25+
<div class="item">
26+
<strong>item 4</strong>
27+
<p>Pellentesque habitant morbi tristique senectus et netus et malesuada fames ac turpis egestas. Vestibulum tortor quam, feugiat vitae, ultricies eget, tempor sit amet, ante. Donec eu libero sit amet quam egestas semper. Aenean ultricies mi vitae est. Mauris placerat eleifend leo. Quisque sit amet est et sapien ullamcorper pharetra. Vestibulum erat wisi, condimentum sed, commodo vitae, ornare sit amet, wisi. Aenean fermentum, elit eget tincidunt condimentum, eros ipsum rutrum orci, sagittis tempus lacus enim ac dui. Donec non enim in turpis pulvinar facilisis. Ut felis.</p>
28+
</div>
29+
30+
<div class="item">
31+
<strong>item 5</strong>
32+
<p>Pellentesque habitant morbi tristique senectus et netus et malesuada fames ac turpis egestas. Vestibulum tortor quam, feugiat vitae, ultricies eget, tempor sit amet, ante. Donec eu libero sit amet quam egestas semper. Aenean ultricies mi vitae est. Mauris placerat eleifend leo. Quisque sit amet est et sapien ullamcorper pharetra. Vestibulum erat wisi, condimentum sed, commodo vitae, ornare sit amet, wisi. Aenean fermentum, elit eget tincidunt condimentum, eros ipsum rutrum orci, sagittis tempus lacus enim ac dui. Donec non enim in turpis pulvinar facilisis. Ut felis.</p>
33+
</div>
34+
35+
<div class="item">
36+
<strong>item 6</strong>
37+
<p>Pellentesque habitant morbi tristique senectus et netus et malesuada fames ac turpis egestas. Vestibulum tortor quam, feugiat vitae, ultricies eget, tempor sit amet, ante. Donec eu libero sit amet quam egestas semper. Aenean ultricies mi vitae est. Mauris placerat eleifend leo. Quisque sit amet est et sapien ullamcorper pharetra. Vestibulum erat wisi, condimentum sed, commodo vitae, ornare sit amet, wisi. Aenean fermentum, elit eget tincidunt condimentum, eros ipsum rutrum orci, sagittis tempus lacus enim ac dui. Donec non enim in turpis pulvinar facilisis. Ut felis.</p>
38+
</div>
39+
40+
<div class="item">
41+
<strong>item 7</strong>
42+
<p>Pellentesque habitant morbi tristique senectus et netus et malesuada fames ac turpis egestas. Vestibulum tortor quam, feugiat vitae, ultricies eget, tempor sit amet, ante. Donec eu libero sit amet quam egestas semper. Aenean ultricies mi vitae est. Mauris placerat eleifend leo. Quisque sit amet est et sapien ullamcorper pharetra. Vestibulum erat wisi, condimentum sed, commodo vitae, ornare sit amet, wisi. Aenean fermentum, elit eget tincidunt condimentum, eros ipsum rutrum orci, sagittis tempus lacus enim ac dui. Donec non enim in turpis pulvinar facilisis. Ut felis.</p>
43+
</div>
44+
45+
<div class="item">
46+
<strong>item 8</strong>
47+
<p>Pellentesque habitant morbi tristique senectus et netus et malesuada fames ac turpis egestas. Vestibulum tortor quam, feugiat vitae, ultricies eget, tempor sit amet, ante. Donec eu libero sit amet quam egestas semper. Aenean ultricies mi vitae est. Mauris placerat eleifend leo. Quisque sit amet est et sapien ullamcorper pharetra. Vestibulum erat wisi, condimentum sed, commodo vitae, ornare sit amet, wisi. Aenean fermentum, elit eget tincidunt condimentum, eros ipsum rutrum orci, sagittis tempus lacus enim ac dui. Donec non enim in turpis pulvinar facilisis. Ut felis.</p>
48+
</div>
49+
50+
<div class="item">
51+
<strong>item 9</strong>
52+
<p>Pellentesque habitant morbi tristique senectus et netus et malesuada fames ac turpis egestas. Vestibulum tortor quam, feugiat vitae, ultricies eget, tempor sit amet, ante. Donec eu libero sit amet quam egestas semper. Aenean ultricies mi vitae est. Mauris placerat eleifend leo. Quisque sit amet est et sapien ullamcorper pharetra. Vestibulum erat wisi, condimentum sed, commodo vitae, ornare sit amet, wisi. Aenean fermentum, elit eget tincidunt condimentum, eros ipsum rutrum orci, sagittis tempus lacus enim ac dui. Donec non enim in turpis pulvinar facilisis. Ut felis.</p>
53+
</div>
54+
55+
<div class="item">
56+
<strong>item 10</strong>
57+
<p>Pellentesque habitant morbi tristique senectus et netus et malesuada fames ac turpis egestas. Vestibulum tortor quam, feugiat vitae, ultricies eget, tempor sit amet, ante. Donec eu libero sit amet quam egestas semper. Aenean ultricies mi vitae est. Mauris placerat eleifend leo. Quisque sit amet est et sapien ullamcorper pharetra. Vestibulum erat wisi, condimentum sed, commodo vitae, ornare sit amet, wisi. Aenean fermentum, elit eget tincidunt condimentum, eros ipsum rutrum orci, sagittis tempus lacus enim ac dui. Donec non enim in turpis pulvinar facilisis. Ut felis.</p>
58+
</div>
59+
</div>
60+
61+
<div class="pagination">
62+
<a href="multiple2.html" class="next">2</a>
63+
</div>
64+
</div>
65+
66+
<div class="overflow2" style="width: 400px; height: 400px; overflow: scroll">
67+
<div class="container">
68+
<div class="item">
69+
<strong>item 1</strong>
70+
<p>Pellentesque habitant morbi tristique senectus et netus et malesuada fames ac turpis egestas. Vestibulum tortor quam, feugiat vitae, ultricies eget, tempor sit amet, ante. Donec eu libero sit amet quam egestas semper. Aenean ultricies mi vitae est. Mauris placerat eleifend leo. Quisque sit amet est et sapien ullamcorper pharetra. Vestibulum erat wisi, condimentum sed, commodo vitae, ornare sit amet, wisi. Aenean fermentum, elit eget tincidunt condimentum, eros ipsum rutrum orci, sagittis tempus lacus enim ac dui. Donec non enim in turpis pulvinar facilisis. Ut felis.</p>
71+
</div>
72+
73+
<div class="item">
74+
<strong>item 2</strong>
75+
<p>Pellentesque habitant morbi tristique senectus et netus et malesuada fames ac turpis egestas. Vestibulum tortor quam, feugiat vitae, ultricies eget, tempor sit amet, ante. Donec eu libero sit amet quam egestas semper. Aenean ultricies mi vitae est. Mauris placerat eleifend leo. Quisque sit amet est et sapien ullamcorper pharetra. Vestibulum erat wisi, condimentum sed, commodo vitae, ornare sit amet, wisi. Aenean fermentum, elit eget tincidunt condimentum, eros ipsum rutrum orci, sagittis tempus lacus enim ac dui. Donec non enim in turpis pulvinar facilisis. Ut felis.</p>
76+
</div>
77+
78+
<div class="item">
79+
<strong>item 3</strong>
80+
<p>Pellentesque habitant morbi tristique senectus et netus et malesuada fames ac turpis egestas. Vestibulum tortor quam, feugiat vitae, ultricies eget, tempor sit amet, ante. Donec eu libero sit amet quam egestas semper. Aenean ultricies mi vitae est. Mauris placerat eleifend leo. Quisque sit amet est et sapien ullamcorper pharetra. Vestibulum erat wisi, condimentum sed, commodo vitae, ornare sit amet, wisi. Aenean fermentum, elit eget tincidunt condimentum, eros ipsum rutrum orci, sagittis tempus lacus enim ac dui. Donec non enim in turpis pulvinar facilisis. Ut felis.</p>
81+
</div>
82+
83+
<div class="item">
84+
<strong>item 4</strong>
85+
<p>Pellentesque habitant morbi tristique senectus et netus et malesuada fames ac turpis egestas. Vestibulum tortor quam, feugiat vitae, ultricies eget, tempor sit amet, ante. Donec eu libero sit amet quam egestas semper. Aenean ultricies mi vitae est. Mauris placerat eleifend leo. Quisque sit amet est et sapien ullamcorper pharetra. Vestibulum erat wisi, condimentum sed, commodo vitae, ornare sit amet, wisi. Aenean fermentum, elit eget tincidunt condimentum, eros ipsum rutrum orci, sagittis tempus lacus enim ac dui. Donec non enim in turpis pulvinar facilisis. Ut felis.</p>
86+
</div>
87+
88+
<div class="item">
89+
<strong>item 5</strong>
90+
<p>Pellentesque habitant morbi tristique senectus et netus et malesuada fames ac turpis egestas. Vestibulum tortor quam, feugiat vitae, ultricies eget, tempor sit amet, ante. Donec eu libero sit amet quam egestas semper. Aenean ultricies mi vitae est. Mauris placerat eleifend leo. Quisque sit amet est et sapien ullamcorper pharetra. Vestibulum erat wisi, condimentum sed, commodo vitae, ornare sit amet, wisi. Aenean fermentum, elit eget tincidunt condimentum, eros ipsum rutrum orci, sagittis tempus lacus enim ac dui. Donec non enim in turpis pulvinar facilisis. Ut felis.</p>
91+
</div>
92+
93+
<div class="item">
94+
<strong>item 6</strong>
95+
<p>Pellentesque habitant morbi tristique senectus et netus et malesuada fames ac turpis egestas. Vestibulum tortor quam, feugiat vitae, ultricies eget, tempor sit amet, ante. Donec eu libero sit amet quam egestas semper. Aenean ultricies mi vitae est. Mauris placerat eleifend leo. Quisque sit amet est et sapien ullamcorper pharetra. Vestibulum erat wisi, condimentum sed, commodo vitae, ornare sit amet, wisi. Aenean fermentum, elit eget tincidunt condimentum, eros ipsum rutrum orci, sagittis tempus lacus enim ac dui. Donec non enim in turpis pulvinar facilisis. Ut felis.</p>
96+
</div>
97+
98+
<div class="item">
99+
<strong>item 7</strong>
100+
<p>Pellentesque habitant morbi tristique senectus et netus et malesuada fames ac turpis egestas. Vestibulum tortor quam, feugiat vitae, ultricies eget, tempor sit amet, ante. Donec eu libero sit amet quam egestas semper. Aenean ultricies mi vitae est. Mauris placerat eleifend leo. Quisque sit amet est et sapien ullamcorper pharetra. Vestibulum erat wisi, condimentum sed, commodo vitae, ornare sit amet, wisi. Aenean fermentum, elit eget tincidunt condimentum, eros ipsum rutrum orci, sagittis tempus lacus enim ac dui. Donec non enim in turpis pulvinar facilisis. Ut felis.</p>
101+
</div>
102+
103+
<div class="item">
104+
<strong>item 8</strong>
105+
<p>Pellentesque habitant morbi tristique senectus et netus et malesuada fames ac turpis egestas. Vestibulum tortor quam, feugiat vitae, ultricies eget, tempor sit amet, ante. Donec eu libero sit amet quam egestas semper. Aenean ultricies mi vitae est. Mauris placerat eleifend leo. Quisque sit amet est et sapien ullamcorper pharetra. Vestibulum erat wisi, condimentum sed, commodo vitae, ornare sit amet, wisi. Aenean fermentum, elit eget tincidunt condimentum, eros ipsum rutrum orci, sagittis tempus lacus enim ac dui. Donec non enim in turpis pulvinar facilisis. Ut felis.</p>
106+
</div>
107+
108+
<div class="item">
109+
<strong>item 9</strong>
110+
<p>Pellentesque habitant morbi tristique senectus et netus et malesuada fames ac turpis egestas. Vestibulum tortor quam, feugiat vitae, ultricies eget, tempor sit amet, ante. Donec eu libero sit amet quam egestas semper. Aenean ultricies mi vitae est. Mauris placerat eleifend leo. Quisque sit amet est et sapien ullamcorper pharetra. Vestibulum erat wisi, condimentum sed, commodo vitae, ornare sit amet, wisi. Aenean fermentum, elit eget tincidunt condimentum, eros ipsum rutrum orci, sagittis tempus lacus enim ac dui. Donec non enim in turpis pulvinar facilisis. Ut felis.</p>
111+
</div>
112+
113+
<div class="item">
114+
<strong>item 10</strong>
115+
<p>Pellentesque habitant morbi tristique senectus et netus et malesuada fames ac turpis egestas. Vestibulum tortor quam, feugiat vitae, ultricies eget, tempor sit amet, ante. Donec eu libero sit amet quam egestas semper. Aenean ultricies mi vitae est. Mauris placerat eleifend leo. Quisque sit amet est et sapien ullamcorper pharetra. Vestibulum erat wisi, condimentum sed, commodo vitae, ornare sit amet, wisi. Aenean fermentum, elit eget tincidunt condimentum, eros ipsum rutrum orci, sagittis tempus lacus enim ac dui. Donec non enim in turpis pulvinar facilisis. Ut felis.</p>
116+
</div>
117+
</div>
118+
119+
<div class="pagination">
120+
<a href="multiple2.html" class="next">2</a>
121+
</div>
122+
</div>
123+
</body>
124+
</html>

0 commit comments

Comments
 (0)