Position "Left / Right / Center"
Align "Left / Right / Center"

It has a different translation in some languages.

CommentFileSizeAuthor
align-context.patch1.04 KBdroplet
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

droplet’s picture

droplet CreditAttribution: droplet commented
Issue tags: +string context

tagging

jamesliu78’s picture

jamesliu78
Primary language Chinese, Traditional
CreditAttribution: jamesliu78 commented

Oh, it's a good idea!!!
Reviewed.

jamesliu78’s picture

jamesliu78
Primary language Chinese, Traditional
CreditAttribution: jamesliu78 commented

I also fined a similar problem in:
core/modules/image/image.admin.inc: Line 449

tky’s picture

tky CreditAttribution: tky commented

Right thing to do.

droplet’s picture

droplet CreditAttribution: droplet commented

@jamesliu78,

probably you found another problem :)
#2037847: Join position string together

jamesliu78’s picture

jamesliu78
Primary language Chinese, Traditional
CreditAttribution: jamesliu78 commented

Oh~that's sounds good. :)

Sutharsan’s picture

Sutharsan CreditAttribution: Sutharsan commented
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community
xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm commented

align-context.patch queued for re-testing.

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.x, thanks!

Status: Fixed » Closed (fixed)

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