Alignment

CommentFileSizeAuthor
#3 639246_axyjo_1.patch358 bytesaxyjo
#3 preview.png9.79 KBaxyjo
#2 639246.patch784 bytesBoobaa
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Issue tags: +Novice

I think this might be easy, so tagging as Novice.

Boobaa’s picture

Boobaa
he/him
CreditAttribution: Boobaa commented
Status: Active » Needs review
FileSize
639246.patch784 bytes

LTR seems to be OK, dunno how to check RTL.

axyjo’s picture

axyjo CreditAttribution: axyjo commented
FileSize
preview.png9.79 KB
639246_axyjo_1.patch358 bytes

Got it.

axyjo’s picture

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

Ignore my patch. Go with boobaa's. It's shorter in the end.

webchick’s picture

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

Wow. That was like the fastest turnaround time on a patch EVER. :D Great work!

I was going to mark needs work because there's no corresponding hunk in styles-rtl.css, only to find that Seven doesn't have a styles-rtl.css. :( Sad. :( So I created #639268: Seven is missing a styles-rtl.css to deal with that.

Committed to HEAD!

Status: Fixed » Closed (fixed)
Issue tags: -Novice

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

Issue tags: +Novice

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.