The node creation page has a couple of RTL issues.

The auto-complete throbber had a 2px whitespace on the right side in the image itself, so I edited the image and added a 2px whitespace to the left also.

LTR
LTR



RTL (before)
RTL (before)



RTL (after)

RTL (after)

CommentFileSizeAuthor
node-create-rtl-fix.patch3.87 KBherom
node-add-rtl-after.png34.88 KBherom
node-create-rtl-before-annotated.png47.69 KBherom
node-add-ltr.png36.36 KBherom
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

eftimitac’s picture

eftimitac CreditAttribution: eftimitac commented

Tested on Chrome(Mac) and IE9(Win7)
So issues described above were fixed,
http://goo.gl/FHE31Q

Can you please fix some other inline issues on this page?
http://goo.gl/Hm9Ahl

herom’s picture

herom CreditAttribution: herom commented

Thanks for testing the patch.

The other issues you pointed will be fixed by #2343281: Fix RTL issues in content listing page, aka /admin/content, I think.

herom’s picture

herom CreditAttribution: herom commented

Closed #2349473: Fixing RTL rotation on node/edit as a duplicate of this.

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented
Status: Needs review » Reviewed & tested by the community

Looks good to me.

DickJohnson’s picture

DickJohnson CreditAttribution: DickJohnson commented

Looks good and fixes the problem.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, node-create-rtl-fix.patch, failed testing.

Status: Needs work » Needs review

DickJohnson queued node-create-rtl-fix.patch for re-testing.

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented
Status: Needs review » Reviewed & tested by the community
Issue tags: +language-base, +sprint
alexpott’s picture

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

This issue is a normal bug fix, and doesn't include any disruptive changes, so it is allowed per https://www.drupal.org/core/beta-changes. Committed 5eee2e7 and pushed to 8.0.x. Thanks!

  • alexpott committed 5eee2e7 on 8.0.x 
    Issue #2329649 by herom: Fix node create page RTL CSS
Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented
Issue tags: -sprint

Superb, thanks!

Status: Fixed » Closed (fixed)

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

jwilson3’s picture

jwilson3
Primary language English
Location Ecuador
CreditAttribution: jwilson3 at Bluespark commented

This issue appears to have caused a regression, that is now RTBC. Would appreciate further look at it from any who participated here:

#2642362: Animation of throbber-active.gif image is broken