Closed Bug 1236296 Opened 4 years ago Closed 4 years ago

Change "View Certificates" to "Manage Certificates" to get to Certificate Manager

Categories

(Thunderbird :: Preferences, defect, minor)

defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 46.0

People

(Reporter: wsmwk, Assigned: Paenglab)

Details

Attachments

(1 file, 1 obsolete file)

Tools | Options | Advanced | Certificates...

I see "View Certificates", and it's not the least bit apparent that I must use "View" to delete or import certificates.
Attached patch manageCerts.patch (obsolete) — Splinter Review
Yes, this makes sense, the dialog title is also "Certificate Manager" and suite has already the Manage Certificate.

I had also to change the entity names in mailnews to trigger the changes for locales but only changed the entity names in suite.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #8703411 - Flags: review?(mkmelin+mozilla)
Comment on attachment 8703411 [details] [diff] [review]
manageCerts.patch

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

r=mkmelin with the 1's change to 2's

::: mail/locales/en-US/chrome/messenger/am-smime.dtd
@@ +25,5 @@
>  <!ENTITY signMessage.accesskey "D">
>  <!ENTITY signingCert.message "Use this certificate to digitally sign messages you send:">
>  
>  <!ENTITY certificates.label "Certificates">
> +<!ENTITY manageCerts1.label "Manage Certificates">

1 is a easily misinterpreted as l, so 2 is preferred.

::: mail/locales/en-US/chrome/messenger/preferences/advanced.dtd
@@ +105,5 @@
>  
> +<!ENTITY manageCertificates.label        "Manage Certificates">
> +<!ENTITY manageCertificates.accesskey    "M">
> +<!ENTITY viewSecurityDevices.label       "Security Devices">
> +<!ENTITY viewSecurityDevices.accesskey   "D">

might want to lose the extra spaces to make things line up, which is a bit futile
Attachment #8703411 - Flags: review?(mkmelin+mozilla) → review+
Addressed the review comments. Carrying the r+ from previous patch.
Attachment #8703411 - Attachment is obsolete: true
Attachment #8703453 - Flags: review+
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/015e0b60011d145b80d6438846736bb34b667273
Bug 1236296 - Change "View Certificates" to "Manage Certificates" to get to Certificate Manager. r=mkmelin a=me
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 46.0
You need to log in before you can comment on or make changes to this bug.