Closed
Bug 372908
Opened 18 years ago
Closed 17 years ago
[fy-NL] migration wizard string change
Categories
(Mozilla Localizations :: fy-NL / Frisian, defect)
Mozilla Localizations
fy-NL / Frisian
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Pike, Assigned: fryskefirefox)
References
Details
(Keywords: fixed1.8.1.4)
Attachments
(1 file, 1 obsolete file)
1.49 KB,
patch
|
Details | Diff | Splinter Review |
We want to change the description of the homepage in the migration wizard,
http://lxr.mozilla.org/l10n-mozilla1.8/source/fy-NL/other-licenses/branding/firefox/brand.properties
It should look like http://lxr.mozilla.org/mozilla/source/other-licenses/branding/firefox/locales/en-US/brand.properties
The corresponding patch is the en-US part in https://bugzilla.mozilla.org/attachment.cgi?id=257411&action=diff#mozilla/other-licenses/branding/firefox/locales/en-US/brand.properties_sec1
Current String in English for homePageSingleStartMain:
"Firefox Start, a fast search page with search results by XXX"
New String:
"Firefox Start, a fast home page with built-in search"
Please create the patch as
cvs -z3 diff -u fy-NL/other-licenses/branding/firefox/brand.properties
from the l10n directory, we'll try to grab those patches together and land them with the en-US change.
See bug 372409 and bug 371309 for more information.
I hope I did this one right, otherwise please let me know.
Wim
Attachment #257662 -
Flags: approval-l10n?
Reporter | ||
Comment 2•18 years ago
|
||
Comment on attachment 257662 [details] [diff] [review]
homePageSingleStartMain string change
approval-l10n is dead, cancelling that.
Note that we'll land the patches we get in one go, so there's currently no need to request approval for those.
On the patch, is there a line ending change going on? And the en-US patch removed quite a few entries, which you're adding back.
Attachment #257662 -
Flags: approval-l10n?
This one must be better.
Wim
Attachment #257662 -
Attachment is obsolete: true
Reporter | ||
Comment 4•18 years ago
|
||
Comment on attachment 257671 [details] [diff] [review]
changed endings and removed lines
Yes, thanks, this one looks good.
You don't need to worry about landing that on the branch, we intend to do that in one go with the en-US patch.
Reporter | ||
Comment 5•18 years ago
|
||
Adding fixed1.8.1.4 keyword as I landed the patch here as part of bug 372409 and attachment 258556 [details] [diff] [review].
Please land this on the trunk yourself and mark this bug fixed, and use the verified1.8.1.4 keyword after verifying my landing on the branch.
Thanks for the patches.
Keywords: fixed1.8.1.4
Reporter | ||
Comment 6•17 years ago
|
||
From what I can see, this bug is FIXED.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•