Closed
Bug 518438
Opened 15 years ago
Closed 7 years ago
[de] Different (false) quotation marks
Categories
(Mozilla Localizations :: de / German, defect)
Mozilla Localizations
de / German
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Thunderbird_Mail_DE, Unassigned)
Details
Attachments
(3 files, 1 obsolete file)
9.08 KB,
patch
|
atopal
:
review+
kairo
:
feedback+
|
Details | Diff | Splinter Review |
2.15 KB,
patch
|
mozilla
:
review+
|
Details | Diff | Splinter Review |
32.14 KB,
patch
|
kairo
:
review+
|
Details | Diff | Splinter Review |
As a follow up to an other bug. I've been searching in all German locale files for different quotation marks. IMHO we should use the quotation mark ". But we're using both " and the apostrophe '. I'm not sure, how to fix this. DTD files are using the " at the beginning and end of every entity. Would it be necessary (and possible?) to encode the quotation marks? AFAIK in DTD files we could use the ' at the beginning and end of an entity, but would this be a practicable solution? KaiRo what does MozillaTranslator with our " quotation marks, when you're saving/exporting the DTD files? In *.properties files the " quotation marks are no (technical) problem.
Comment 1•15 years ago
|
||
We need to use " in DTDs, and I think it should work everywhere.
Comment 2•14 years ago
|
||
Makes sense, di you want to take this, Alexander?
Reporter | ||
Comment 3•14 years ago
|
||
(In reply to comment #2) > Makes sense, di you want to take this, Alexander? Taking over
Assignee: nobody → AlexIhrig
Reporter | ||
Comment 4•14 years ago
|
||
Attachment #503466 -
Flags: review?
Attachment #503466 -
Flags: feedback?
Reporter | ||
Updated•14 years ago
|
Attachment #503466 -
Flags: review? → review?(a.topal)
Reporter | ||
Updated•14 years ago
|
Attachment #503466 -
Flags: feedback? → feedback?(kairo)
Reporter | ||
Comment 5•14 years ago
|
||
FYI: The patch has a lot of "hidden" changes, because of removing spaces at the end of lines.
Comment 6•14 years ago
|
||
Comment on attachment 503466 [details] [diff] [review] Patch v1 - excluding venkman and inspector It looks like almost half of the changes are those 'hidden' changes, but the other half looks good ;)
Attachment #503466 -
Flags: review?(a.topal) → review+
Comment 7•14 years ago
|
||
Comment on attachment 503466 [details] [diff] [review] Patch v1 - excluding venkman and inspector whatever. will require a lot of work to get my in-MT stuff synched up with this, but what's this world without such fun, right? ;-) I don't like the changes that seem to apply something to comments where the actual change can't be seen here, please don't apply those or explain what they actually are. If they e.g. introduce non-unix linebreaks or such (can't tell from just looking at Bugzilla) then they're inacceptable, if they clean up such things, they probably should be done in their own bug.
Attachment #503466 -
Flags: feedback?(kairo) → feedback+
Reporter | ||
Comment 8•14 years ago
|
||
1) No linebreak changes (UltraEdits pref is set to save edited files in the original format) 2) The only change in these hidden line changes are deleted spaces/tabs at line endings
Comment 9•14 years ago
|
||
(In reply to comment #8) > 2) The only change in these hidden line changes are deleted spaces/tabs at line > endings Couldn't spot those in the example(s) I looked at in the patch - still, my tools will probably add them back in files I manage/edit if they are present in the original en-US versions as well. And I still am not sure if it's a good idea to check them in mixed between the actual changes here.
Reporter | ||
Comment 10•14 years ago
|
||
Patch v2 removes all "hidden" changes. Separate patches for calendar and suite will follow.
Attachment #503466 -
Attachment is obsolete: true
Attachment #503568 -
Flags: review?
Reporter | ||
Updated•14 years ago
|
Attachment #503568 -
Flags: review? → review?(a.topal)
Reporter | ||
Updated•14 years ago
|
Attachment #503568 -
Flags: feedback?(kairo)
Comment 11•14 years ago
|
||
Comment on attachment 503568 [details] [diff] [review] [checked in] Patch v2 - excluding calendar, suite and venkman/inspector Much better :)
Attachment #503568 -
Flags: review?(a.topal) → review+
Reporter | ||
Comment 12•14 years ago
|
||
Attachment #503668 -
Flags: review?
Reporter | ||
Updated•14 years ago
|
Attachment #503668 -
Flags: review? → review?(mozrob)
Reporter | ||
Comment 13•14 years ago
|
||
Attachment #503669 -
Flags: review?(kairo)
Reporter | ||
Updated•14 years ago
|
Attachment #503568 -
Attachment description: Patch v2 - excluding calendar, suite and venkman/inspector → [checked in] Patch v2 - excluding calendar, suite and venkman/inspector
Updated•14 years ago
|
Attachment #503669 -
Flags: review?(kairo) → review+
Reporter | ||
Comment 14•13 years ago
|
||
Fx part checked in: http://hg.mozilla.org/l10n-central/de/rev/3057612f2161 Suite part checked in: http://hg.mozilla.org/l10n-central/de/rev/b1548f13abd7
Reporter | ||
Updated•13 years ago
|
Attachment #503669 -
Attachment description: Patch v2 - suite part → [checked in] Patch v2 - suite part
Reporter | ||
Comment 15•13 years ago
|
||
Missing replacement in suite part: http://hg.mozilla.org/l10n-central/de/rev/ff94642c47bf
Reporter | ||
Comment 16•13 years ago
|
||
In venkman and chatzilla I can find many ' and '' and ``. I'm not sure if a part of these signs are used for formating purposes in chatzilla(s GUI). Additional there are some strings beginning with ''%S without an ending quotation?! So this part should be done by Robert.
Comment 17•13 years ago
|
||
(In reply to comment #16) > In venkman and chatzilla I can find many ' and '' and ``. I'm not sure if a > part of these signs are used for formating purposes in chatzilla(s GUI). > Additional there are some strings beginning with ''%S without an ending > quotation?! So this part should be done by Robert. I followed what the original strings have there, and with the strange L10n stuff they have, I have no clue what of those they actually need or not, so I don't really dare to touch that stuff.
Comment 18•13 years ago
|
||
Comment on attachment 503668 [details] [diff] [review] [checked in] Patch v2 - calendar part My first patch to review is a hard one! ;-) Thanks, Alexander.
Attachment #503668 -
Flags: review+
Reporter | ||
Updated•13 years ago
|
Attachment #503668 -
Flags: review?(mozrob)
Reporter | ||
Comment 19•13 years ago
|
||
Calendar part checked in: http://hg.mozilla.org/l10n-central/de/rev/fa0d1e7bc92d Venkman and chatzilla are remaining, other parts are fixed.
Assignee: AlexIhrig → nobody
Reporter | ||
Updated•13 years ago
|
Attachment #503668 -
Attachment description: Patch v2 - calendar part → [checked in] Patch v2 - calendar part
Updated•13 years ago
|
Attachment #503568 -
Flags: feedback?(kairo) → feedback+
Comment 20•7 years ago
|
||
opi123, please top changing the status of bugs - at least most of them are still valid.
Flags: needinfo?(niklasbetz20)
Comment 21•7 years ago
|
||
I'm sorry, but for me it is as it should be. So what should I mark the bug as?
You need to log in
before you can comment on or make changes to this bug.
Description
•