Closed
Bug 219208
Opened 21 years ago
Closed 21 years ago
duplicate entities in messengercompose.dtd
Categories
(SeaMonkey :: MailNews: Message Display, defect)
SeaMonkey
MailNews: Message Display
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: Martijn.Ras, Assigned: Martijn.Ras)
Details
Attachments
(1 file, 1 obsolete file)
|
258 bytes,
patch
|
sspitzer
:
review+
mscott
:
superreview+
|
Details | Diff | Splinter Review |
Heya Folks,
Both the label and accesskey of the printSetupCmd are duplicated in
messengercompose.dtd.
Mazzel,
Martijn.
|
Assignee | |
Comment 1•21 years ago
|
||
|
|
Assignee | |
Comment 2•21 years ago
|
||
Comment on attachment 131453 [details]
Removes one of the duplicated labels and accesskeys for the printSetupCmd.
--- locale/en-US/messenger/messengercompose/messengercompose.dtd Sun Sep 14
19:17:03 2003
+++ messengercompose.dtd Sun Sep 14 19:17:16 2003
@@ -43,2 +42,0 @@
-<!ENTITY printSetupCmd.label "Pagina-instellingen...">
-<!ENTITY printSetupCmd.accesskey "I">
Hi Martin -
Can you remove these instead/as well? -
/mailnews/compose/resources/locale/en-US/messengercompose.dtd, line 38 --
<!ENTITY printSetupCmd.label "Page Setup...">
/mailnews/compose/resources/locale/en-US/messengercompose.dtd, line 43 --
<!ENTITY printSetupCmd.label "Page Setup...">
and
/mailnews/compose/resources/locale/en-US/messengercompose.dtd, line 39 --
<!ENTITY printSetupCmd.accesskey "u">
/mailnews/compose/resources/locale/en-US/messengercompose.dtd, line 44 --
<!ENTITY printSetupCmd.accesskey "u">
(Your patch is regionalized for the Netherlands).
|
|
Assignee | |
Comment 5•21 years ago
|
||
Sorry, got too many sources around here ... should have figured i'd used the
wrong one when i had to 'tweak' the patch ...
|
|
Assignee | |
Updated•21 years ago
|
Attachment #131453 -
Attachment is obsolete: true
Attachment #131453 -
Attachment is patch: false
|
||
Updated•21 years ago
|
Attachment #131456 -
Flags: superreview?(scott)
Attachment #131456 -
Flags: review?(sspitzer)
|
|
||
Comment 6•21 years ago
|
||
Assigning to you, Martijn, since you have the patch.
Assignee: sspitzer → Martijn.Ras
|
||
Updated•21 years ago
|
Attachment #131456 -
Flags: superreview?(scott) → superreview+
|
||
Comment 7•21 years ago
|
||
Comment on attachment 131456 [details] [diff] [review]
Dit maal in de juiste taal ...
r=sspitzer
Attachment #131456 -
Flags: review?(sspitzer) → review+
|
|
||
Comment 8•21 years ago
|
||
Timeless landed the patch on the trunk.
Thanks for the patch, Martijn!
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
|
||
Comment 9•21 years ago
|
||
Note: next time, please diff -u3 at the very least (-u9 for big patches).
Verified fixed using LXR.
Status: RESOLVED → VERIFIED
|
||
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•