There are a few issues in the content listing page. There is a patch, and screenshots.

LTR



RTL - before



RTL - after

CommentFileSizeAuthor
#5 interdiff-content-listing-rtl.txt737 bytesupchuk
#5 content-listing-rtl-2343281-5.patch2.84 KBupchuk
content-listing-rtl.patch2.84 KBherom
content-listing-rtl-fixed.png36.35 KBherom
content-listing-rtl-before-annotated.png61.97 KBherom
content-listing-ltr.png35.92 KBherom

Comments

herom’s picture

herom commented
Issue summary: View changes
eftimitac’s picture

eftimitac commented

Verified on IE9 and FF(Win7), Chrome and Safari on Mac
Looks fine, you can review screenshots:
http://goo.gl/wKlnQ3
http://goo.gl/M2YdwM

lewisnyman’s picture

lewisnyman
he/him
commented
Status: Needs review » Needs work

Looking really good

+++ b/core/modules/system/css/system.theme.css
@@ -401,18 +401,22 @@ ul.links a.active {
+[dir="rtl"] .breadcrumb ol {
+  /* this is required to win over specifity of [dir="rtl"] ol */

@@ -455,18 +459,22 @@ ul.tabs {
+[dir="rtl"] .action-links {
+  /* this is required to win over specifity of [dir="rtl"] ul */

I think we just need to capitalise these comments.

upchuk’s picture

upchuk commented
Assigned: Unassigned » upchuk

I can do that real quick.

upchuk’s picture

upchuk commented
Assigned: upchuk » Unassigned
Status: Needs work » Needs review
StatusFileSize
new content-listing-rtl-2343281-5.patch2.84 KB
new interdiff-content-listing-rtl.txt737 bytes

Here you go.

D

lewisnyman’s picture

lewisnyman
he/him
commented
Status: Needs review » Reviewed & tested by the community

Thank you!

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Committed 4691167 and pushed to 8.0.x. Thanks!

  • alexpott committed 4691167 on 8.0.x 
    Issue #2343281 by Upchuk, herom: Fixed RTL issues in content listing...

Status: Fixed » Closed (fixed)

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