Closed
Bug 435550
Opened 15 years ago
Closed 13 years ago
about: flawed for localization
Categories
(Firefox :: General, defect)
Tracking
()
RESOLVED
FIXED
Firefox 3.7a5
People
(Reporter: glandium, Assigned: glandium)
Details
Attachments
(1 file, 2 obsolete files)
2.80 KB,
patch
|
Details | Diff | Splinter Review |
Currently, phrases in about: are cut in three strings: beforeLink, linkTitle and afterLink. While this is not a bad idea, what is one is to have a space "hardcoded" between each of those. Some languages, such as japanese, usually don't use spaces in sentences. It happens about.dtd is not localized in japanese, but if it were, the about: page would look weird.
Assignee | ||
Comment 1•15 years ago
|
||
Assignee | ||
Updated•15 years ago
|
Attachment #322348 -
Flags: review? → review?(l10n)
Comment 2•15 years ago
|
||
Comment on attachment 322348 [details] [diff] [review] patch r-, this needs either a change in entity names or a patch for l10n, l10n is frozen right now, though. Probably both. We shouldn't really touch this until we have l10n for Firefox 3.1 up for general consumption, which will take a while.
Attachment #322348 -
Flags: review?(l10n) → review-
Comment 3•15 years ago
|
||
Attachment #322348 -
Attachment is obsolete: true
Attachment #344007 -
Flags: review?(l10n)
Comment 4•15 years ago
|
||
Comment on attachment 344007 [details] [diff] [review] patch - v2 I don't see my review comments addressed, r- for the same reasons as before.
Attachment #344007 -
Flags: review?(l10n) → review-
Comment 5•15 years ago
|
||
Comment on attachment 344007 [details] [diff] [review] patch - v2 I changed the entity names. Look closer. :)
Attachment #344007 -
Flags: review- → review?(l10n)
Comment 6•15 years ago
|
||
Oops, sorry. CCing Adrian and Gandalf, would you have cycles to create a silme script to generate l10n patches corresponding the en-US one? If, we could either publish the script for localizers, or the patches, possibly landing the latter, too.
Comment 7•15 years ago
|
||
working on that
Updated•14 years ago
|
Attachment #344007 -
Flags: review?(l10n) → review+
Comment 8•14 years ago
|
||
Comment on attachment 344007 [details] [diff] [review] patch - v2 r=me, though I'm not eager to land this right away. I really don't think this is big enough to break string freeze, and for landing on central, this could lead to confusion then. If you don't mind holding back on it 'til "sometime next year" ;-)
Updated•13 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 10•13 years ago
|
||
Attachment #344007 -
Attachment is obsolete: true
Assignee | ||
Updated•13 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 11•13 years ago
|
||
Before I check that in (now that I can), is there some prerequisite for l10n or is it just okay to change strings on m-c ?
Keywords: checkin-needed
Comment 12•13 years ago
|
||
I guess we should just land this, but make the check-in comment say explicitly that you're moving the whitespace from the xhtml to the DTD, so that people reading that know what they want to do. Also, could you add a localization note similar to the one that's in the context to this patch? And once you land, a quick post to mozilla.dev.l10n about that change would be good, for those that don't read the check-in comments.
Assignee | ||
Comment 13•13 years ago
|
||
Thanks, will do when m-c reopens.
Assignee | ||
Comment 14•13 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/d10d49ee252d
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.7a5
You need to log in
before you can comment on or make changes to this bug.
Description
•