Opened 10 years ago
Closed 10 years ago
#26181 closed defect (bug) (fixed)
RTL: Login screen has 92% for padding-top
Reported by: | ocean90 | Owned by: | nacin |
---|---|---|---|
Milestone: | 3.8 | Priority: | normal |
Severity: | minor | Version: | |
Component: | RTL | Keywords: | needs-patch |
Focuses: | ui | Cc: |
Description
See http://cl.ly/SdZQ.
Attachments (1)
Change History (9)
#2
in reply to:
↑ 1
@
10 years ago
Thanks for the ping, Dominik. I'll look into it - maybe a bug in the nodejs module we're using.
#5
@
10 years ago
The same happens for .theme
. The module transforms margin: 0 5% 4% 0;
into margin: 0 0 96% 5%
.
#6
@
10 years ago
Bug filed upstream: https://github.com/trevorparscal/cssjanus/pull/4
The cause is that the Background flipping regex is too greedy causing it to incorrectly flip the Margins.
#7
@
10 years ago
I have a fork at https://github.com/nacin/cssjanus that incorporates fixes by both dd32 and yoavf. Going to "pin" some things for now.
Note: See
TracTickets for help on using
tickets.
Adding
/* @noflip */
seems to fix the issue for now, see 26181.patch. I have no idea why it's swapped. Maybe yoavf has something.For the record: I can't reproduce the issue with http://cssjanus.commoner.com/.