Closed
Bug 807699
Opened 12 years ago
Closed 11 years ago
Sanitize dialog ("Clear Recent History") cleanup: unused strings
Categories
(Thunderbird :: Mail Window Front End, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 23.0
People
(Reporter: aryx, Assigned: Cykesiopka)
Details
Attachments
(1 file, 1 obsolete file)
2.89 KB,
patch
|
Details | Diff | Splinter Review |
Thunderbird 17.0 Beta (buildid 20121030125442) on Windows XP SP3 32 bit The "Clear Recent History" menu item in the "Tools" menu launches a dialog. There are some issues which are small and can all be fixed in one patch: 1. The accesskeys and the listbox don't work, either make me them work or remove them: http://hg.mozilla.org/releases/comm-beta/diff/76efc33d143a/mail/base/content/sanitize.xul 2. There are unused strings, use them or remove them: <!ENTITY sanitizePrefs2.title "Settings for Clearing History"> <!ENTITY sanitizeItems.label "Clear the following items now:"> (this one is also not used in /browser/) <!ENTITY clearDataSettings2.label "When I quit &brandShortName;, it should automatically clear all:"> <!ENTITY clearTimeDuration.dateColumn "Visit Date"> <!ENTITY clearTimeDuration.nameColumn "Name"> <!ENTITY historySection.label "History"> <!ENTITY dataSection.label "Data">
Comment 1•12 years ago
|
||
perhaps a different bug, but does this really even deserve to be in a thunderbird menu?
Updated•12 years ago
|
Component: General → Mail Window Front End
Assignee | ||
Comment 2•11 years ago
|
||
I have no idea what to do about the listbox or the access keys, so just the string removal part for now...
Attachment #744972 -
Flags: review?(mbanner)
Comment 3•11 years ago
|
||
(In reply to Wayne Mery (:wsmwk) from comment #1) > perhaps a different bug, but does this really even deserve to be in a > thunderbird menu? Yes, it was a requirement from the privacy folks/review.
Comment 4•11 years ago
|
||
Comment on attachment 744972 [details] [diff] [review] Proposed Part 1: String Removal v1 Looks great, r=Standard8. At a quick glance, I don't see why the access keys wouldn't work and Firefox has the same issue as well. I suggest we file a core bug on the access keys issue, and close this one (once the patch has landed) as the removing unused strings.
Attachment #744972 -
Flags: review?(mbanner) → review+
Assignee | ||
Comment 5•11 years ago
|
||
I changed the commit message so it doesn't mention "Part 1" anymore, as this bug is now only about the string removal.
Assignee: nobody → cykesiopka.bmo
Attachment #744972 -
Attachment is obsolete: true
Status: NEW → ASSIGNED
Assignee | ||
Updated•11 years ago
|
Keywords: checkin-needed
Summary: Sanitize dialog ("Clear Recent History") cleanup: unused strings, inaccessible accesskeys → Sanitize dialog ("Clear Recent History") cleanup: unused strings
Comment 6•11 years ago
|
||
https://hg.mozilla.org/comm-central/rev/d7c72b841e66
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 23.0
Assignee | ||
Comment 7•11 years ago
|
||
(In reply to Mark Banner (:standard8) from comment #4) > I suggest we file a core bug on the access keys issue I filed Bug 870615 for this.
You need to log in
before you can comment on or make changes to this bug.
Description
•