Add context for translators [#1952872]

Comment	File	Size	Author
#5	menu_editor-add-t-context-5.patch	914 bytes	Sutharsan
#3	menu_editor-add-t-context-2.patch	953 bytes	Sutharsan
#2	menu_editor-add-t-context-2.patch	953 bytes	Sutharsan

Comment #1

donquixote CreditAttribution: donquixote commented 26 March 2013 at 12:31

I will be happy for a patch!
Maybe you find more places than this.

Log in or register to post comments

I don't mind making this small patch, if you give credit in your commit message as described in Commit messages - providing history and credit. For example: #1952872 by Sutharsan: Context added to t() for translation of abbreviations.

Log in or register to post comments

Comment #3

Sutharsan CreditAttribution: Sutharsan commented 27 March 2013 at 10:56

File	Size
menu_editor-add-t-context-2.patch	953 bytes

[edit] Deleting duplicate comment.

Log in or register to post comments

Comment #4

donquixote CreditAttribution: donquixote commented 27 March 2013 at 11:04

Thanks!
I usually try, sometimes forget..
But usually I will make it so that the person really shows as the commit author, whereas the comment only has the issue link.
Your name will suit this module nicely :)

'abbreviation Expanded'

is this standard practice for abbreviations, or just the closest you came up with?
I don't really mind but it is good to know :)

Log in or register to post comments

Comment #5

Sutharsan CreditAttribution: Sutharsan commented 27 March 2013 at 16:28

File	Size
menu_editor-add-t-context-5.patch	914 bytes

I did not know you can change the commit author, but I see that the contributions get counted equally.

There is currently limited use of context (some 20 - 30 contexts defined). Personally I think because the barrier is too high. It requires investigation, creating a patch and discussing with maintainers ( ;) but true). This takes a lot of time and requires developer skills. So the current use is limited to draw any conclusions on what the standard practice is. I researched a little more on the current use of context. Most contexts are less verbose, so I did cut it down a bit.

Log in or register to post comments

Comment #6

donquixote CreditAttribution: donquixote commented 27 March 2013 at 20:43

Ok, I am tempted to commit like this, but I am hesitating a bit by the noise that each translation change causes for translators.

It is unclear to me whether the context is meant for the human or for the machine.
Documentation is poor: http://api.drupal.org/api/drupal/includes%21bootstrap.inc/function/t/7