config_translation.admin.css does not include RTL-specific css. Right now, the source column is at left, and the destination column is at right, against the right-to-left flow in an RTL page.

Before patch:


After patch:

Files: 

Comments

Status: Needs review » Needs work

The last submitted patch, config-trans-admin-rtl.patch, failed testing.

Status: Needs work » Needs review

herom queued config-trans-admin-rtl.patch for re-testing.

Gábor Hojtsy’s picture

Looks great. Is the responsive version correct as well? When you size the window less wide? Is the source above the translation still?

herom’s picture

FileSize
22.41 KB

YES, it is!

seiplax’s picture

Assigned: Unassigned » seiplax
Issue tags: +FUDK
seiplax’s picture

Assigned: seiplax » Unassigned
Status: Needs review » Reviewed & tested by the community
FileSize
52.76 KB
54.95 KB

Tested on 8.0.x

Verified by enabling Configuration translations module that ships with D8 Core. The direct path with Arabic installed is admin/structure/types/manage/article/translate/ar/edit

Gábor Hojtsy’s picture

Issue tags: +D8MI, +language-config

Yay. I was concerned the responsive version would not work, but yay :)

seiplax’s picture

FileSize
50.9 KB

yes, also confirming that mobile works.

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Good catch, and awesome mobile testing, too! :D

Committed and pushed to 8.x. Thanks!

  • webchick committed 0025ee4 on 8.0.x
    Issue #2315459 by herom, seiplax: Fixed RTL for config translation form.
    

Status: Fixed » Closed (fixed)

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