Change text for easier translation [#42520]

This patch does following:

-      return t('<p>Enter the name for your new menu. Remember to enable the newly created block in the %blocks administration page.</p>', array('%blocks' => l(t('blocks'), 'admin/block')));
+      return t('<p>Enter the name for your new menu. Remember to enable the newly created block in the %blocks.</p>', array('%blocks' => l(t('blocks administration page'), 'admin/block')));

As it was, it was impossibile in polish to translate it correctly. For english this patch doesn't change anything.

Comment	File	Size	Author
#6	menu_translation_0.diff	1.07 KB	Cvbge
#6
	menu_translation.diff	1.06 KB	Cvbge

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

Dries CreditAttribution: Dries commented 28 December 2005 at 12:06

Looks like Cvbge is in a patch-frenzy! Rock on! Committed to HEAD. Thanks.

Comment #2

Dries CreditAttribution: Dries commented 28 December 2005 at 12:08

Status:

Needs review

» Fixed

Comment #3

killes@www.drop.org CreditAttribution: killes@www.drop.org commented 28 December 2005 at 12:36

Status:

Fixed

» Active

Hmm, I did discuss this patch with Cvbge on IRC and we agreed that having a href in the help text (as in the other help texts) would be better.

Comment #4

Cvbge CreditAttribution: Cvbge commented 28 December 2005 at 13:59

Indeed, didn't think this would be committed so fast ;)
We could use e.g.

t('<p>Enter the name for your new menu. Remember to enable the newly created block in the <a href="admin/block">blocks administration page</a>.</p>');

but I don't think current solution is much worse (although it's worse a bit, because provides smaller translation possibilities)...

Should I make a new patch?