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.

Sign up for GitHub

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

Jump to bottom

Accordion: Attempt at improving animation. #587

Closed
wants to merge 11 commits into from
Closed

Accordion: Attempt at improving animation. #587

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
d9db3c9
Accordion: Attempt at improving animation.
scottgonzalez Jan 31, 2012
90d1c78
Merge branch 'master' into accordion-animation
scottgonzalez Feb 9, 2012
fe4d1e3
Merge branch 'master' into accordion-animation
scottgonzalez Feb 9, 2012
fab80df
Accordion: Cleaned up animation code.
scottgonzalez Feb 9, 2012
1565602
Merge branch 'master' into accordion-animation
scottgonzalez Feb 11, 2012
779c45b
Accordion: Cleaned up height equalization.
scottgonzalez Feb 12, 2012
3462ce2
Accordion: New animation API.
scottgonzalez Feb 12, 2012
7460356
Merge branch 'master' into accordion-animation
scottgonzalez Feb 12, 2012
0f43dc4
Accordion: Fixed animation for heightStyle: content; Force height of …
scottgonzalez Feb 12, 2012
af94c37
Accordion: Changed option name to animate and added back-compat for a…
scottgonzalez Feb 16, 2012
edb9411
Merge branch 'master' into accordion-animation
scottgonzalez Feb 16, 2012
File filter

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Accordion: Cleaned up height equalization.
  • Loading branch information
@scottgonzalez
scottgonzalez committed Feb 12, 2012
commit 779c45bf65918bd1f5f12fe7089eb2308e1b31c4
50 changes: 26 additions & 24 deletions ui/jquery.ui.accordion.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -432,18 +432,20 @@ $.widget( "ui.accordion", {
$.extend( $.ui.accordion, {
animations: {
slide: function( options ) {
var total;

if ( !options.toHide.size() ) {
return options.toShow.animate( showProps, options );
}
if ( !options.toShow.size() ) {
return options.toHide.animate( hideProps, options );
}

var total = options.toShow.show().outerHeight();
total = options.toShow.show().outerHeight();
options.toHide.animate( hideProps, options );
options.toShow
.hide()
.data( "togglePair", {
.data( "accordionHeight", {
total: total,
toHide: options.toHide
})
Expand All @@ -452,21 +454,35 @@ $.extend( $.ui.accordion, {
easing: options.easing,
complete: function() {
setTimeout(function() {
options.toShow.removeData( "togglePair" );
options.toShow.removeData( "accordionHeight" );
options.complete();
}, 1 );
}
});
},
bounceslide: function( options ) {
this.slide( $.extend( options, {
easing: options.down ? "easeOutBounce" : "swing",
duration: options.down ? 1000 : 200
}));
}
},
bounceslide: function( options ) {
this.slide( $.extend( options, {
easing: options.down ? "easeOutBounce" : "swing",
duration: options.down ? 1000 : 200
}));
}
}
});

$.fx.step.accordionHeight = function( fx ) {
var elem = $( fx.elem ),
data = elem.data( "accordionHeight" );
elem.height( data.total - elem.outerHeight() - data.toHide.outerHeight() + elem.height() );
};
var hideProps = {},
showProps = {},
showPropsAdjust = {};
hideProps.height = hideProps.paddingTop = hideProps.paddingBottom =
hideProps.borderTopWidth = hideProps.borderBottomWidth = "hide";
showProps.height = showProps.paddingTop = showProps.paddingBottom =
showProps.borderTopWidth = showProps.borderBottomWidth = "show";
$.extend( showPropsAdjust, showProps, { accordionHeight: "show" } );



// DEPRECATED
Expand Down Expand Up @@ -611,18 +627,4 @@ if ( $.uiBackCompat !== false ) {
}( jQuery, jQuery.ui.accordion.prototype ) );
}

$.fx.step._height = function( fx ) {
var elem = $( fx.elem ),
data = elem.data( "togglePair" );
elem.height( data.total - elem.outerHeight() - data.toHide.outerHeight() + elem.height() );
};
var hideProps = {},
showProps = {},
showPropsAdjust = {};
hideProps.height = hideProps.paddingTop = hideProps.paddingBottom =
hideProps.borderTopWidth = hideProps.borderBottomWidth = "hide";
showProps.height = showProps.paddingTop = showProps.paddingBottom =
showProps.borderTopWidth = showProps.borderBottomWidth = "show";
$.extend( showPropsAdjust, showProps, { _height: "show" } );

})( jQuery );