Hey,
here is a small location-rtl.css file - for RTL support :)

my suggestions:

  • remove the float to have the label on top of the input field as in the rest of the system.
  • remove the text-align, since it is inherited anyway from the rest of the system's css.
    RTL themes will have text-align:right ans LTR will have text-align:left (somewhere closer to the body element)
CommentFileSizeAuthor
#9 Location-RTL-support-1151374-10685950.patch376 bytesrreiss
#7 Location-RTL-support-1151374-10685950.patch0 bytesrreiss
#5 Location-RTL-1151374-#5.patch379 bytesrreiss
#3 Location-RTL-1151374-#3.patch415 bytesrreiss
#1 location-rtl.txt112 bytesmtk

Comments

mtk’s picture

mtk commented
StatusFileSize
new location-rtl.txt112 bytes
bnadem’s picture

bnadem commented

Thanks a lot, you saved me from a terrible headache, as I tried to fix it using the theme overriding local system inspite of the module's theming "location" itself !
Thank you again for sharing your solution ;)

rreiss’s picture

rreiss commented
Version: 6.x-3.1 » 7.x-3.7
Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new Location-RTL-1151374-#3.patch415 bytes

Still lack of support in RTL. Patch is attached.
Please review and commit to dev.

Thanks.

Status: Needs review » Needs work

The last submitted patch, 3: Location-RTL-1151374-#3.patch, failed testing.

rreiss’s picture

rreiss commented
Version: 7.x-3.7 » 7.x-3.x-dev
Status: Needs work » Needs review
StatusFileSize
new Location-RTL-1151374-#5.patch379 bytes

Status: Needs review » Needs work

The last submitted patch, 5: Location-RTL-1151374-#5.patch, failed testing.

rreiss’s picture

rreiss commented
Status: Needs work » Needs review
StatusFileSize
new Location-RTL-support-1151374-10685950.patch0 bytes

Status: Needs review » Needs work

The last submitted patch, 7: Location-RTL-support-1151374-10685950.patch, failed testing.

rreiss’s picture

rreiss commented
Status: Needs work » Needs review
StatusFileSize
new Location-RTL-support-1151374-10685950.patch376 bytes
podarok’s picture

podarok
🇺🇦 he/him/his
Primary language Ukrainian
Location Ukraine
commented
Status: Needs review » Fixed

Thanks, merged.

  • podarok committed e989d42 on 7.x-3.x authored by rreiss 
    Issue #1151374 by rreiss, mtk: Location - RTL

Status: Fixed » Closed (fixed)

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