Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.- 'left-top' => t('Top') . ' ' . t('Left'),
- 'center-top' => t('Top') . ' ' . t('Center'),
- 'right-top' => t('Top') . ' ' . t('Right'),
- 'left-center' => t('Center') . ' ' . t('Left'),
Not every language could combine 2 word in this way :)
eg:
In Eng, you would say:
Top Left
In other language it is:
Left Top
| Comment | File | Size | Author |
|---|---|---|---|
| #9 | 2037847-join-string-d7.patch | 1.21 KB | droplet |
| #4 | 2037847-join-string.patch | 1.4 KB | droplet |
Comments
Comment #0.0
droplet commentedUpdated issue summary.
Comment #0.1
droplet commentedUpdated issue summary.
Comment #1
amourowAgreed, it's better described in a string for each spot in this case and good for translation.
Comment #1.0
amourowUpdated issue summary.
Comment #2
droplet commentedfix_string.patch queued for re-testing.
Comment #4
droplet commentedsame patch, reroll only
Comment #5
droplet commentedComment #6
droplet commented4: 2037847-join-string.patch queued for re-testing.
Comment #7
penyaskitoReviewed and looks a nice fix. I run grep for checking that there are not more occurrences of this after this patch, and does not look like.
Comment #8
webchickWow. That is utterly bizarre.
Git blame led me to #371374: Add ImageCache UI Core so this was part of the initial patch. No idea how I missed it back in 2009. :) I chalk it up to excitement about having an image cache UI in core. :D
Committed and pushed to 8.x. Thanks!
This can likely be backported to 7.x as well.
Comment #9
droplet commentedComment #10
dawehnerJust wonder whether we should also remove the unnecessary whitespaces automatically.
Comment #11
izus commentedi tested path #9,
it applied well and the result looks good for me.
Thanks
Comment #13
longwave9: 2037847-join-string-d7.patch queued for re-testing.
Comment #14
longwaveAbove fail was an auto retest of an RTBC patch, failure was a bot fluke (probably disk space)
Comment #16
droplet commented9: 2037847-join-string-d7.patch queued for re-testing.
Comment #17
droplet commentedTest bots issues only, back to RTBC
Comment #19
David_Rothstein commentedCommitted to 7.x - thanks!
Comment #20
David_Rothstein commentedDrupal 7.27 was a security release, so this is now scheduled to go out in Drupal 7.28 instead.