system,messages-rtl.css is missing.
I am not sure yet how to add files to a patch but here is the file :
[had to add a .txt extension to upload, just remove it]

CommentFileSizeAuthor
#3 system.messages-rtl.patch537 bytestsi
#3 messages-chrome.png9.07 KBtsi
#3 messages-firefox.png9.68 KBtsi
#3 messages-ie7.png7.72 KBtsi
#3 messages-ie8.png7.76 KBtsi
#1 system.messages-rtl.patch567 bytestsi
system.messages-rtl.css_.txt217 bytestsi

Comments

tsi’s picture

tsi commented
StatusFileSize
new system.messages-rtl.patch567 bytes

Here is a patch -

aspilicious’s picture

aspilicious commented

1) You don't write LTR in a rtl patch you have to do that in the normal css file
2) Can we have some crossbrowser screenshots?

tsi’s picture

tsi commented
Issue tags: +RTL
StatusFileSize
new messages-ie8.png7.76 KB
new messages-ie7.png7.72 KB
new messages-firefox.png9.68 KB
new messages-chrome.png9.07 KB
new system.messages-rtl.patch537 bytes

1) Obviously
2) Here they are, with a new patch :

aspilicious’s picture

aspilicious commented

Where is the system.messages.css part?

Otherwise looks good :)

tsi’s picture

tsi commented

No change to system.messages.css (already has /* LTR */ comments).
Thanks

aspilicious’s picture

aspilicious commented
Status: Needs review » Reviewed & tested by the community

double checked :)

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Committed to HEAD. Thanks!

Status: Fixed » Closed (fixed)
Issue tags: -RTL

Automatically closed -- issue fixed for 2 weeks with no activity.

Issue tags: +RTL

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.