improve L12y of FAYT statusbar messages

NEW
Unassigned

Status

defect
11 years ago
10 years ago

People

(Reporter: kairo, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

11 years ago
From bug 345526 comment #29:

>+  showStatusMatch: function(aResult, aExtra) {
>+    // Set the status text from a find result
>+    // link|text "..." [not] found [next|previous match] [(href)]
>+    if (aExtra)
>+      aExtra = " " + this.mBundle.GetStringFromName(aExtra);
>+    var url = "";
>+    var string = this.mLinks ? "link" : "text";
>+    if (aResult == Components.interfaces.nsITypeAheadFind.FIND_NOTFOUND)
>+      string += "not";
>+    else if (this.mFind.foundLink && this.mFind.foundLink.href)
>+      url = "  " + this.mBundle.GetStringFromName("openparen") +
>+                   this.mFind.foundLink.href +
>+                   this.mBundle.GetStringFromName("closeparen");
>+    string += "found";
>+    this.showStatus(this.mBundle.GetStringFromName(string) +
>+                    this.mSearchString +
>+                    this.mBundle.GetStringFromName("closequote") +
>+                    aExtra + url);
>+  },

This looks rather messy to me from a L10N point of view. Shouldn't this be one
L10N string with placeholders to be filled in, because $language could have
weird ideas about word order and such?
You need to log in before you can comment on or make changes to this bug.