1305385 - Remove section about Edit button from Peoples tab and Server tab in the Certificate Manager

Status: RESOLVED FIXED

(SeaMonkey :: Help Documentation, defect)

Description

[Onno Ekker [:nONoNonO UTC+1]]

While localizing certs_help.xhtml I came across the help for the Edit button on the Peoples tab, which I couldn't find in the UI. It seems that this functionality was removed in bug 1130418, but the corresponding help text was never updated.

Comment 1

[Onno Ekker [:nONoNonO UTC+1]]

In bug 825583 the Edit functionality was also removed from the Server tab…

Comment 2

[Onno Ekker [:nONoNonO UTC+1]]

Attachment: Remove Edit sections from certs_help.xhtml

Comment 3

[Stefan [:stefanh]]

Comment on attachment 8795252 [details] [diff] [review]
Remove Edit sections from certs_help.xhtml

Thanks for the patch. Note that you should always fill in the requestee field when you set the review flag to "?". Then the requestee will get an email (having a blank requestee field is risky, your request might not get any attention).

Anyway, I've looked at your patch and it looks good :-)

One thing, though: The "Import" button in the Servers tab doesn't seem to exist anymore, so please remove that from the help file as well.

If you put up a new patch with that change, I'll r+ it

Comment 4

[Stefan [:stefanh]]

(In reply to Stefan [:stefanh] from comment #3) 
> If you put up a new patch with that change, I'll r+ it

If you ask me for review, of course ;-)

Comment 5

[Onno Ekker [:nONoNonO UTC+1]]

Attachment: Patch v2

I did see two other nits: in bug 173729 the Edit and Delete buttons on the Authorities tab were renamed to Edit Trust and Delete or Distrust. I think it suffices to only change the button's labels and that the text itself doesn't need to change…

Comment 6

[Stefan [:stefanh]]

Comment on attachment 8796008 [details] [diff] [review]
Patch v2

Thanks!

Comment 7

[Stefan [:stefanh]]

https://hg.mozilla.org/comm-central/rev/9569b194aa19