355967 - $ signs in WARN_APP_RUNNING_INSTALL

Status: RESOLVED FIXED

(Mozilla Localizations :: el / Greek, defect)

Description

[Marek Stępień [:marcoos, inactive]]

This is the same problem as reported for Polish in bug 355966.

Probably while migrating strings from commonLocale.nsh to custom.properties, the Greek teamsomehow didn't notice the .nsh-ish endlines: $\r$\n$\r$\n in
WARN_APP_RUNNING_INSTALL. 

This makes the dollar signs visible in the window with that error message.

The endlines should be changed to just "\n\n".

Comment 1

[Marek Stępień [:marcoos, inactive]]

Affected files:

http://lxr.mozilla.org/l10n-mozilla1.8/source/el/browser/installer/custom.properties#66
http://lxr.mozilla.org/l10n-mozilla1.8/source/el/mail/installer/custom.properties#65

Comment 2

[Marek Stępień [:marcoos, inactive]]

Attachment: Patch for browser

Fixing browser, as this WFMs in mail now.

pkst, you may want to have this checked-in for 2.0.0.1. 

You should also request approval1.8.1.1 from Pike for checking this in.

Comment 3

[Kostas Papadimas]

Comment on attachment 244839 [details] [diff] [review]
Patch for browser

Marek thanks  for the point and the patch. Requesting  approval for 1.8.1.1

Comment 4

[Axel Hecht [:Pike]]

Comment on attachment 244839 [details] [diff] [review]
Patch for browser

[el] approving installer fix for firefox 2.0.0.1

Please use the fixed1.8.1.1 and verified1.8.1.1 keywords to track landing and testing.

Comment 5

[Marek Stępień [:marcoos, inactive]]

You may want to fix that on trunk as well.