The problem occur when i enable forum (core forum) and open main forum page (for example www.site-name/forum). The page is in RTL language (hebrew) but the text alignment is left. I suppose that problem can occur not only in forum main page. In order to fix it you need to add to style-rtl.css file the next lines of code:
tr td,
tr th {
padding: 4px 9px;
border: 1px solid #fff;
text-align: right; /* RTL */ /* fix */
}

regards,

Itzhak

CommentFileSizeAuthor
#19 bartik_style_rtl-1320028-d7-19.patch371 bytesalbert volkman
#17 bartik_style_rtl-1320028-d7-17.patch362 bytesalbert volkman
#14 bartik-style-rtl-1320028-14.patch412 bytesisay
#7 bartik-style-rtl-1320028-7.patch491 bytesisay
#5 bartik-style-rtl-1320028-5.patch511 bytesisay
#3 bartik-style-rtl-1320028-3.patch491 bytesisay

Comments

isay’s picture

isay commented
Assigned: isay » Unassigned
isay’s picture

isay commented
Version: 7.8 » 7.x-dev
isay’s picture

isay commented
Version: 7.x-dev » 8.x-dev
Assigned: Unassigned » isay
Status: Active » Needs review
StatusFileSize
new bartik-style-rtl-1320028-3.patch491 bytes

I found that the same problem exist in 8.x version so submitting patch for 8.x version. Is after the patch will be tested the backporting to 7.x will be done automatically? Or should I submit the same patch for 7.x version?
I am submitting patch for the first time so I hope I did it right.

Status: Needs review » Needs work

The last submitted patch, bartik-style-rtl-1320028-3.patch, failed testing.

isay’s picture

isay commented
Status: Needs work » Needs review
StatusFileSize
new bartik-style-rtl-1320028-5.patch511 bytes

Status: Needs review » Needs work

The last submitted patch, bartik-style-rtl-1320028-5.patch, failed testing.

isay’s picture

isay commented
Version: 8.x-dev » 7.x-dev
Status: Needs work » Needs review
StatusFileSize
new bartik-style-rtl-1320028-7.patch491 bytes
isay’s picture

isay commented

#5: bartik-style-rtl-1320028-5.patch queued for re-testing.

isay’s picture

isay commented
Version: 7.x-dev » 8.x-dev
isay’s picture

isay commented

#5: bartik-style-rtl-1320028-5.patch queued for re-testing.

isay’s picture

isay commented
Component: theme system » Bartik theme

this patch belongs to bartik theme

isay’s picture

isay commented
Status: Needs review » Reviewed & tested by the community
aspilicious’s picture

aspilicious commented
Status: Reviewed & tested by the community » Needs work 
+++ b/themes/bartik/css/style-rtl.cssundefined
@@ -17,6 +17,15 @@ blockquote:after {
+  padding: 4px 9px;
+  border: 1px solid #fff;

This is not needed in the rtl file

The other line is ok but remove the /* LTR */ comment

-23 days to next Drupal core point release.

isay’s picture

isay commented
Status: Needs work » Needs review
StatusFileSize
new bartik-style-rtl-1320028-14.patch412 bytes

change the patch by aspilicious's recommendation

aspilicious’s picture

aspilicious commented
Status: Needs review » Reviewed & tested by the community
catch’s picture

catch
he/him
Primary language English
commented
Version: 8.x-dev » 7.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Looks fine. Committed/pushed to 8., moving back to 8.x.

albert volkman’s picture

albert volkman commented
Status: Patch (to be ported) » Needs review
StatusFileSize
new bartik_style_rtl-1320028-d7-17.patch362 bytes

7.x, even :) Backport!

aspilicious’s picture

aspilicious commented
Status: Needs review » Needs work

You're missing the "tr td"

albert volkman’s picture

albert volkman commented
Status: Needs work » Needs review
StatusFileSize
new bartik_style_rtl-1320028-d7-19.patch371 bytes

Arggh.

aspilicious’s picture

aspilicious commented
Status: Needs review » Reviewed & tested by the community

good :)

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 7.x. Thanks!

Status: Fixed » Closed (fixed)

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

cweagans’s picture

cweagans
He/Him
Primary language English
Location Boise, ID, USA
commented
Issue tags: +Needs backport to D7

Updating tags per http://drupal.org/node/1517250

albert volkman’s picture

albert volkman commented
Issue tags: -Needs backport to D7

Already was backported :)