Closed
Bug 420537
Opened 16 years ago
Closed 16 years ago
Remove unusable printAttachmentCmd strings
Categories
(Thunderbird :: Mail Window Front End, defect)
Thunderbird
Mail Window Front End
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 3
People
(Reporter: philor, Assigned: philor)
References
()
Details
Attachments
(2 files)
1.26 KB,
patch
|
mkmelin
:
review+
|
Details | Diff | Splinter Review |
1.37 KB,
patch
|
neil
:
review+
neil
:
superreview+
|
Details | Diff | Splinter Review |
Bug 75691 added printAttachmentCmd.label and printAttachmentCmd.accesskey unused, but either due to a typo or as a hidden file joke, used ".Print" with a leading dot. Now we have about half the locales using a dot, and half not, so if someone ever implements attachment printing, the string will be unusable - it'll take a new name to get the dotted half to realize they should drop the dot, and even then it's not a sure thing. Better to just trash it, which will get everyone to start clean when someone does implement it.
Attachment #306831 -
Flags: review?(mkmelin+mozilla)
Assignee | ||
Comment 1•16 years ago
|
||
^
Attachment #306832 -
Flags: superreview?(neil)
Attachment #306832 -
Flags: review?(neil)
Comment 2•16 years ago
|
||
Comment on attachment 306831 [details] [diff] [review] Tb fix [checked in] Always good to clean up unused stuff! r=me
Attachment #306831 -
Flags: review?(mkmelin+mozilla) → review+
Assignee | ||
Comment 3•16 years ago
|
||
Comment on attachment 306831 [details] [diff] [review] Tb fix [checked in] mail/locales/en-US/chrome/messenger/msgHdrViewOverlay.dtd 1.12
Attachment #306831 -
Attachment description: Tb fix → Tb fix [checked it]
Updated•16 years ago
|
Attachment #306832 -
Flags: superreview?(neil)
Attachment #306832 -
Flags: superreview+
Attachment #306832 -
Flags: review?(neil)
Attachment #306832 -
Flags: review+
Assignee | ||
Comment 4•16 years ago
|
||
Comment on attachment 306832 [details] [diff] [review] SM fix [checked in] suite/locales/en-US/chrome/mailnews/msgHdrViewOverlay.dtd 1.20
Attachment #306832 -
Attachment description: SM fix → SM fix [checked in]
Assignee | ||
Updated•16 years ago
|
Attachment #306831 -
Attachment description: Tb fix [checked it] → Tb fix [checked in]
Assignee | ||
Updated•16 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 5•16 years ago
|
||
(In reply to comment #0) > unused, but either due to a typo or as a hidden file joke, used ".Print" According to timeless, neither one: in the way-back days, that was actually the way to say that the string was currently unused. Makes me wonder if any others have survived.
You need to log in
before you can comment on or make changes to this bug.
Description
•