Skip to content

Commit 6e29977

Browse files
author
Maximum Hallinan
committed
Add destroy method to bigSlideAPI
1 parent ea8472d commit 6e29977

File tree

2 files changed

+7
-5
lines changed

2 files changed

+7
-5
lines changed

src/jquery.big-slide.js

+5-4
Original file line numberDiff line numberDiff line change
@@ -77,8 +77,8 @@
7777
},
7878

7979
// remove bigSlide behavior from the menu
80-
destroy: function(){
81-
view.destroy();
80+
_destroy: function(){
81+
view._destroy();
8282

8383
delete menuLink.bigSlideAPI;
8484

@@ -161,7 +161,7 @@
161161
}
162162
},
163163

164-
destroy: function(){
164+
_destroy: function(){
165165
//remove inline styles generated by bigSlide.js while preserving any other inline styles
166166
this.$menu.each(function(){
167167
var $this = $(this);
@@ -211,7 +211,8 @@
211211
settings: settings,
212212
model: model,
213213
controller: controller,
214-
view: view
214+
view: view,
215+
destroy: controller._destroy
215216
};
216217

217218
return this;

test/big-slide.spec.js

+2-1
Original file line numberDiff line numberDiff line change
@@ -110,6 +110,7 @@ describe('exposed internal components', function(){
110110
expect($menuLink.bigSlideAPI.model).toBeDefined();
111111
expect($menuLink.bigSlideAPI.controller).toBeDefined();
112112
expect($menuLink.bigSlideAPI.view).toBeDefined();
113+
expect($menuLink.bigSlideAPI.destroy).toBeDefined();
113114
});
114115

115116
it('properties of the exposed settings object should have expected values', function(){
@@ -154,7 +155,7 @@ describe('destroy method', function(){
154155

155156
$menuLink.trigger('click.bigSlide');
156157

157-
$menuLink.bigSlideAPI.controller.destroy();
158+
$menuLink.bigSlideAPI.destroy();
158159
});
159160

160161
it('the jQuery object should not have a bigSlide property', function(){

0 commit comments

Comments
 (0)