90192 - fix localization notes in search-attributes.properties

Status: RESOLVED WORKSFORME

(MailNews Core :: Localization, defect)

Description

[(not reading, please use [e-mail] instead)]

please fix the localization notes in in search-attributes.properties
> # LOCALIZATION NOTE FILE: Translate only items 0 to 10
> # Other items may be used in version 6.1 or later

it doesn't look like 0 to 10 is correct.  for example, "Age In Days" should be
localized, it is #24

I'm not sure about #9 (common name).

#10 and up seem like they should not be localized, but again I'm not sure how
(or if) they are used.  at first glance they look like LDAP attributes which
would not be visible to the user.

Comment 1

[Håkan Waara]

Common name should not be localized, hence my move of the item in bug 90165

I propose you move the LOCALIZATION NOTE to have it below all the items that
should be localized and then write something like "LOCALIZATION NOTE: Translate
only the items above. Others may be used in a later version of the application."

That way you don't hardcode the app's name (Netscape/Mozilla) and not the
numbers of the items.

Comment 2

[ji]

Reassigned to rchen, since Allan is not working here anymore.

Comment 3

[Serge Gautherie (:sgautherie)]

The old comments were replaced in
<http://bonsai.mozilla.org/cvslog.cgi?file=mozilla/suite/locales/en-US/chrome/mailnews/search-attributes.properties&rev=HEAD&mark=1.8>

Is there anything else to do here ?

Comment 4

[Phil Ringnalda (:philor)]

No.