Skip to content

Effects: set right/bottom to 'auto' so effects work with dir=rtl. Fixed #6736 - Bug in slide effect when dir=rtl and has position.right #70

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.

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

Closed
wants to merge 9 commits into from
2 changes: 1 addition & 1 deletion ui/jquery.effects.blind.js
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
2 changes: 1 addition & 1 deletion ui/jquery.effects.bounce.js
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
2 changes: 1 addition & 1 deletion ui/jquery.effects.clip.js
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
2 changes: 1 addition & 1 deletion ui/jquery.effects.core.js
Original file line number Diff line number Diff line change
Expand Up @@ -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();
Expand Down
2 changes: 1 addition & 1 deletion ui/jquery.effects.drop.js
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
2 changes: 1 addition & 1 deletion ui/jquery.effects.fold.js
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
4 changes: 2 additions & 2 deletions ui/jquery.effects.scale.js
Original file line number Diff line number Diff line change
Expand Up @@ -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'];
Expand Down
2 changes: 1 addition & 1 deletion ui/jquery.effects.shake.js
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
2 changes: 1 addition & 1 deletion ui/jquery.effects.slide.js
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down