Skip to content
Permalink
Browse files

Effects: set right/bottom to 'auto' so effects work with dir=rtl. Fix…

…ed #6736 - Bug in slide effect when dir=rtl and has position.right.
  • Loading branch information...
allpro authored and scottgonzalez committed Dec 13, 2010
1 parent f73d421 commit 208454934439482cdbef8ba67a8557a799d44abb
@@ -17,7 +17,7 @@ $.effects.blind = function(o) {
return this.queue(function() {

// Create element
var el = $(this), props = ['position','top','left'];
var el = $(this), props = ['position','top','bottom','left','right'];

// Set options
var mode = $.effects.setMode(el, o.options.mode || 'hide'); // Set Mode
@@ -17,7 +17,7 @@ $.effects.bounce = function(o) {
return this.queue(function() {

// Create element
var el = $(this), props = ['position','top','left'];
var el = $(this), props = ['position','top','bottom','left','right'];

// Set options
var mode = $.effects.setMode(el, o.options.mode || 'effect'); // Set Mode
@@ -17,7 +17,7 @@ $.effects.clip = function(o) {
return this.queue(function() {

// Create element
var el = $(this), props = ['position','top','left','height','width'];
var el = $(this), props = ['position','top','bottom','left','right','height','width'];

// Set options
var mode = $.effects.setMode(el, o.options.mode || 'hide'); // Set Mode
@@ -390,7 +390,7 @@ $.extend($.effects, {
props[pos] = 'auto';
}
});
element.css({position: 'relative', top: 0, left: 0 });
element.css({position: 'relative', top: 0, left: 0, right: 'auto', bottom: 'auto' });
}

return wrapper.css(props).show();
@@ -17,7 +17,7 @@ $.effects.drop = function(o) {
return this.queue(function() {

// Create element
var el = $(this), props = ['position','top','left','opacity'];
var el = $(this), props = ['position','top','bottom','left','right','opacity'];

// Set options
var mode = $.effects.setMode(el, o.options.mode || 'hide'); // Set Mode
@@ -17,7 +17,7 @@ $.effects.fold = function(o) {
return this.queue(function() {

// Create element
var el = $(this), props = ['position','top','left'];
var el = $(this), props = ['position','top','bottom','left','right'];

// Set options
var mode = $.effects.setMode(el, o.options.mode || 'hide'); // Set Mode
@@ -84,8 +84,8 @@ $.effects.size = function(o) {
return this.queue(function() {

// Create element
var el = $(this), props = ['position','top','left','width','height','overflow','opacity'];
var props1 = ['position','top','left','overflow','opacity']; // Always restore
var el = $(this), props = ['position','top','bottom','left','right','width','height','overflow','opacity'];
var props1 = ['position','top','bottom','left','right','overflow','opacity']; // Always restore
var props2 = ['width','height','overflow']; // Copy for children
var cProps = ['fontSize'];
var vProps = ['borderTopWidth', 'borderBottomWidth', 'paddingTop', 'paddingBottom'];
@@ -17,7 +17,7 @@ $.effects.shake = function(o) {
return this.queue(function() {

// Create element
var el = $(this), props = ['position','top','left'];
var el = $(this), props = ['position','top','bottom','left','right'];

// Set options
var mode = $.effects.setMode(el, o.options.mode || 'effect'); // Set Mode
@@ -17,7 +17,7 @@ $.effects.slide = function(o) {
return this.queue(function() {

// Create element
var el = $(this), props = ['position','top','left'];
var el = $(this), props = ['position','top','bottom','left','right'];

// Set options
var mode = $.effects.setMode(el, o.options.mode || 'show'); // Set Mode

0 comments on commit 2084549

Please sign in to comment.
You can’t perform that action at this time.