Open Bug 467004 Opened 16 years ago Updated 16 years ago

improve L12y of FAYT statusbar messages

Categories

(SeaMonkey :: UI Design, defect)

defect
Not set
normal

Tracking

(Not tracked)

People

(Reporter: kairo, Unassigned)

References

Details

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.