Currently, the date field is hardcoded to be displayed on the left side.

The attached file - date-rtl.css - moves the date field to the right for RTL languages.
(Popup date selector is not supported yet).

Please add it to the main module directory as a new file. (No need to change any PHP code - RTL is done automatically at the theme level by RTL-aware themes).

CommentFileSizeAuthor
#17 date-n346752-17.interdiff.txt355 bytesdamienmckenna
#17 date-n346752-17.patch341 bytesdamienmckenna
#14 RTL_Support-346752-10685940.patch339 bytesrreiss
#9 RTL_support-1219236-#8.patch358 bytesrreiss
#3 D6_date_rtl_support2.patch2.76 KBhass

Comments

druvision’s picture

druvision commented

Appearently, I can't attach CSS files, so here it is.

.container-inline-date .form-item .form-item {
  float:right;
}

.container-inline-date  {
  margin-left:0.5em;
  margin-right:0px;
}


.container-inline-date .form-item input,
.container-inline-date .form-item select,
.container-inline-date .form-item option {
  margin-right:0px;
  margin-left:5px;
}

.container-inline-date .date-spacer {
  margin-left:0px;
  margin-right:-5px;
}
druvision’s picture

druvision commented
Status: Reviewed & tested by the community » Needs review
hass’s picture

hass commented
Version: 6.x-2.0-rc6 » 6.x-2.x-dev
StatusFileSize
new D6_date_rtl_support2.patch2.76 KB

Patch attached.

karens’s picture

karens commented
Status: Needs review » Fixed

Thanks, committed.

hass’s picture

hass commented

Only to note - D5 don't have RTL detection... :-)

karens’s picture

karens commented

Oops, I thought it was :) OK, removed that one.

hass’s picture

hass commented

@levavie: Could you take a look if we need to add a datepicker-rtl.css, please?

Status: Fixed » Closed (fixed)

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

rreiss’s picture

rreiss commented
Version: 6.x-2.x-dev » 7.x-2.9
Issue summary: View changes
Status: Closed (fixed) » Needs review
StatusFileSize
new RTL_support-1219236-#8.patch358 bytes

Still minor issue on D7 version.
Patch is attached.

Status: Needs review » Needs work

The last submitted patch, 9: RTL_support-1219236-#8.patch, failed testing.

hass’s picture

hass commented
Version: 7.x-2.9 » 7.x-2.x-dev
Status: Needs work » Needs review

The last submitted patch, 3: D6_date_rtl_support2.patch, failed testing.

Status: Needs review » Needs work

The last submitted patch, 9: RTL_support-1219236-#8.patch, failed testing.

rreiss’s picture

rreiss commented
StatusFileSize
new RTL_Support-346752-10685940.patch339 bytes
eyal shalev’s picture

eyal shalev
he/him
commented
Status: Needs work » Needs review
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Parent issue: » #2867810: Plan for Date 7.x-2.11 release
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
StatusFileSize
new date-n346752-17.patch341 bytes
new date-n346752-17.interdiff.txt355 bytes

A minor tweak.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Fixed

Committed. Thanks.

Status: Fixed » Closed (fixed)

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