Closed Bug 1628799 Opened 4 years ago Closed 4 years ago

Remove unused / duplicate entities from imported editor locale files

Categories

(Thunderbird :: Message Compose Window, task)

task
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 78.0

People

(Reporter: iannbugzilla, Assigned: iannbugzilla)

References

Details

Attachments

(1 file)

When bug 1571687 landed, there was some duplication created between entities in the existing messengercompose locale files and the ones imported from editor.

As far as I can see most of these are duplicated entities, the only ones that I believe are unused are pasteAs.label/accesskey:
https://searchfox.org/comm-central/search?q=pasteAs.&case=false&regexp=false&path=

I would just consider this as part of the fix, there may be other files too but I don't have the time to look through it all. Perhaps if there are can they can be spun off into separate bugs.

Assignee: nobody → iann_bugzilla
Status: NEW → ASSIGNED
Attachment #9139539 - Flags: feedback?(khushil324)

The pasteAs.label/accesskey seemed to have stopped being used somewhere between mozilla 1.8 and mozilla 1.9 releases (14ish years ago).

Comment on attachment 9139539 [details] [diff] [review]
Remove unused / duplicate entities - Part 1

Thanks! Looks good to me. I will look into other files of /messengercompose in another bug.
Attachment #9139539 - Flags: feedback?(khushil324) → feedback+
Attachment #9139539 - Attachment description: WIP remove unused / duplicate entities - Part 1 → Remove unused / duplicate entities - Part 1
Attachment #9139539 - Flags: review?(mkmelin+mozilla)
Comment on attachment 9139539 [details] [diff] [review]
Remove unused / duplicate entities - Part 1

Review of attachment 9139539 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM, r=mkmelin
Attachment #9139539 - Flags: review?(mkmelin+mozilla) → review+
Target Milestone: --- → Thunderbird 78.0

Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/4f44b29f8b4f
Remove unused / duplicate entities from imported editor locale files. r=mkmelin DONTBUILD

Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: