node links are not left aligned in bartik theme on RTL direction [#1387572]

While using RTL direction, when displaying a node links in the bottom are not aligned on the left.
Here is the problem:
display link error
This is the fix:
display link fix

Comment	File	Size	Author
#5	rtl-display-link-1387572-drupal8.patch	337 bytes	pbull
#5
#2	rtl-display-link-1387572-2.patch	317 bytes	pbull
#2
#1	rtl-display-link-1387572-1.patch	316 bytes	pbull
#1
#1	2012-01-06_2341.png	26.05 KB	pbull
	rtl-display-link.patch	360 bytes	tarekdj

	display-link-fix.png	25.61 KB	tarekdj
	display-link-error.png	28.58 KB	tarekdj

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

pbull CreditAttribution: pbull commented 7 January 2012 at 04:43

File	Size
2012-01-06_2341.png	26.05 KB
rtl-display-link-1387572-1.patch	316 bytes

Patch applies cleanly and fixes the RTL issue in FF 7, Chrome 16.

However in Safari 5, longer lists of links wrap prematurely, as seen here:
Safari 5 RTL wrapping

Replacing the proposed CSS with .link-wrapper { text-align:left; } fixes this across all tested browsers. Re-rolling and attaching revised patch.

Comment #2

pbull CreditAttribution: pbull commented 7 January 2012 at 04:46

File	Size
rtl-display-link-1387572-2.patch	317 bytes

Fixing whitespace.

Comment #3

tarekdj CreditAttribution: tarekdj commented 9 January 2012 at 16:21

Status:

Needs review

» Reviewed & tested by the community

Patch works fine! seems to be RTBC for me

Comment #4

xjm

she/her

English

CreditAttribution: xjm commented 17 January 2012 at 19:34

Version:	7.x-dev	» 8.x-dev
Status:	Reviewed & tested by the community	» Needs work
Issue tags:		+Novice, +Needs backport to D7

Thanks for the patch! This needs to be fixed in Drupal 8 first per the backport policy. (We'll fix it in Drupal 7 around the same time.) Could we get a version of the patch rolled against Drupal 8?