search

google / closure-stylesheets

A CSS+ transpiler that lints, optimizes, and I18n-izes
Apache License 2.0
312 stars 65 forks source link

When RTL flipping, border-radius should be treated differenrtly #120

Open udonmai opened 7 years ago

udonmai commented 7 years ago

Now border-radius: 10px 10px 0 0; will be flipped to border-radius: 0 0 10px 10px; while it should be border-radius: 10px 10px 0 0;

since the four values are top-left 1 top-right 2 bottom-left 4 bottom-right 3 -> 1 2 3 4

and should be turned to be 2 1 3 4 -> 2 1 4 3

seanmfoy commented 7 years ago

I used this input:

p {
  border-radius: 10px 10px 0 0;
}
div {
  border-radius: 1px 2px 3px 4px;
}

with this invocation: java -jar target/clsoure-stylesheets-1.5.0-SNAPSHOT-jar-with-dependencies.jar --pretty-print --output-orientation RTL 120.css

and got this output:

p {
  border-radius: 10px 10px 0 0;
}
div {
  border-radius: 2px 1px 4px 3px;
}

Can you give a more detailed recipe to reproduce this problem?