Line breaks done with \n in Javascript strings will cause problems in translation as Pootle escapes them automatically. The result is that in a translated template \n is shown instead of a line break.
Created attachment 36972 [details] [review] Bug 13861: \n in confirmation message on closing a budget \n in strings doesn't translate well, so we move it outside the translated string. To test: - create at least 2 budgets - close one of the budgets (moving orders from one budget to the other) - after selecting the budget to move your orders to, save - the confirmation message shown should read: You have chosen to move all unreceived orders from 'budget 1' to 'budget 2'. This action cannot be reversed. Do you wish to continue?
Created attachment 37004 [details] [review] Bug 13861: \n in confirmation message on closing a budget \n in strings doesn't translate well, so we move it outside the translated string. To test: - create at least 2 budgets - close one of the budgets (moving orders from one budget to the other) - after selecting the budget to move your orders to, save - the confirmation message shown should read: You have chosen to move all unreceived orders from 'budget 1' to 'budget 2'. This action cannot be reversed. Do you wish to continue? Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Very trivial: Passed QA in one run.
Patch pushed to master. Thanks Katrina!
Pushed to 3.18.x will be in 3.18.6