Closed Bug 420537 Opened 16 years ago Closed 16 years ago

Remove unusable printAttachmentCmd strings

Categories

(Thunderbird :: Mail Window Front End, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 3

People

(Reporter: philor, Assigned: philor)

References

()

Details

Attachments

(2 files)

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)
^
Attachment #306832 - Flags: superreview?(neil)
Attachment #306832 - Flags: review?(neil)
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+
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]
Attachment #306832 - Flags: superreview?(neil)
Attachment #306832 - Flags: superreview+
Attachment #306832 - Flags: review?(neil)
Attachment #306832 - Flags: review+
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]
Attachment #306831 - Attachment description: Tb fix [checked it] → Tb fix [checked in]
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
(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.

Attachment

General

Created:
Updated:
Size: