Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Swapping clear property direction #2

Merged
merged 1 commit into from

2 participants

@esten

I'm an engineer on the international team at Twitter, and was assigned to fix a bug involving our new profile pages on the RTL version of the site. Noticed that "clear" wasn't having its direction reversed, so I added it to the list of properties swapped by R2.

@mzsanford mzsanford merged commit b91f87e into from
@mzsanford
Owner

Thanks for the pull request. Glad to see @twitter is still using it :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Oct 8, 2012
  1. @esten
This page is out of date. Refresh to see the latest.
Showing with 2 additions and 1 deletion.
  1. +2 −1  lib/r2.rb
View
3  lib/r2.rb
@@ -54,7 +54,8 @@ class Swapper
'box-shadow' => lambda {|obj,val| obj.quad_swap(val) },
'-webkit-box-shadow' => lambda {|obj,val| obj.quad_swap(val) },
'-moz-box-shadow' => lambda {|obj,val| obj.quad_swap(val) },
- 'direction' => lambda {|obj,val| obj.direction_swap(val) }
+ 'direction' => lambda {|obj,val| obj.direction_swap(val) },
+ 'clear' => lambda {|obj,val| obj.side_swap(val) }
}
# Given a String of CSS perform the full directionality change
Something went wrong with that request. Please try again.