Delete Cert dialogs need to be differentiated, text cleaned up

VERIFIED FIXED in Future

Status

P2
normal
VERIFIED FIXED
18 years ago
2 years ago

People

(Reporter: cotter, Assigned: rangansen)

Tracking

1.0 Branch
Future
x86
Windows NT

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

18 years ago
When you click the Delete button on any Cert Mgr tab, you see this text:

-------
You have requested to delete this certificate:

[cert subject name]

Are you sure you want to delete this certificate?
-------

This is a regression from PSM 1.x, which told you something about the 
consequences 
of deleting whatever kind of cert was involved. In addition, it makes no 
allowance for selecting multiple certs (only the first one shows up).

To match 1.x, some of the text needs to vary depending what kinds of certs you 
are trying to delete. 

In addition to the text changes described below, the dialog should list all 
certs that the user selected before clicking Delete.

Ideally, the title of the dialog should indicate what kind of cert is being 
deleted: Delete Your Certificate, Delete Web Site Certificate, or Delete CA 
Certificate. If this is too difficult, "Delete Certificates" would be OK for all 
three kinds of deletes. (Currently, the title of the dialog is the subject name 
of the cert, which doesn't hold up for multiple selections.)

Otherwise, I recommend that the text for each version look like this:

****Delete Your Certificates

Are you sure you want to delete these certificates?

[list of subject names]

If you delete one of your own certificates, you can no longer use it to identify 
yourself.

****Delete Web Site Certificates

Are you sure you want to delete these web site certificates?

[list of subject names]

If you delete a web site certificate, you will be asked to accept it again the 
next time you visit the web site.

****Delete CA Certificates

Are you sure you want to delete these CA certificates?

[list of subject names]

If you delete a certificate authority (CA) certificate, your browser will no 
longer trust any certificates issued by that CA.
(Reporter)

Comment 1

18 years ago
Changed target to 2.0.

Note also that these dialogs are supposed to have help buttons (ssuming that 
context-sensitivity stuff gets checked in soon). The targets are:

?delete_my_certs
?delete_web_certs
?delete_ca_certs
Target Milestone: --- → 2.0

Comment 2

18 years ago
Mass reassigning target to 2.1
Target Milestone: 2.0 → 2.1

Comment 3

18 years ago
Marking new.
Status: UNCONFIRMED → NEW
Ever confirmed: true

Updated

18 years ago
Keywords: nsenterprise

Comment 4

18 years ago
p2
Priority: -- → P2

Comment 5

18 years ago
rangansen
Assignee: mcgreer → rangansen
(Assignee)

Updated

18 years ago
Status: NEW → ASSIGNED

Updated

18 years ago
Target Milestone: 2.1 → Future

Comment 6

18 years ago
removing nsenterprise keyword from PSM bugs with target milestone of future.
Keywords: nsenterprise

Comment 7

18 years ago
Mass assigning QA to ckritzer.
QA Contact: junruh → ckritzer
(Assignee)

Comment 8

17 years ago
Here is a patch that fixes this problem. However, this too, includes the kind of
strange workaround mention in bug# 94384
(Assignee)

Comment 9

17 years ago
Created attachment 45140 [details] [diff] [review]
patch

Comment 10

17 years ago
A couple of comments:

1) You could consider passing a nsIPKIParamBlock instead of a
nsIDialogParamBlock.  This allows you to pass nsISupports objects as well
(meaning you can pass a reference to the cert instead of just the certdbkey and
then have to look it up again.)  This is just for consideration, take or leave
it, either way.

These should get fixed:
1)In the <window... definition, avoid using a hard-coded width.  Use something
like style="width: 8ex" instead.  This causing the width to be relative to the
font size.

2) Add the absolute path for these JavaScript files and add the string
'type="application/x-javascript" to them as well.
 
<script src="deletecert.js"/>
+  <script type="application/x-javascript"
src="chrome://global/content/strres.js" />
+  <script src="pippki.js" />
<script type="application/x-javascript" src="chrome://help/content/help.js" />

3) Don't add 'style= "width.."' strings to buttons.  (Bug 88328)

Fix the above 3, and you have r=javi.
(Assignee)

Comment 11

17 years ago
Created attachment 45311 [details] [diff] [review]
patch
(Assignee)

Comment 12

17 years ago
Done ...

Comment 13

17 years ago
sr=hewitt
(Assignee)

Comment 14

17 years ago
Patch checked in.
I would also like to point out that the window, being persistent, remembers the
older value, and might show sizing issues first time, depending on the profile
used [bug# 94755]. This might be resized, though.
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED

Comment 15

17 years ago
Marking VERIFIED FIXED on:
-MacOS91 2001-08-15-08-trunk (commercial)
-MacOS_X 2001-08-15-05-trunk (commercial)
-LinRH62 2001-08-15-08-trunk (commercial)
-Win98SE 2001-08-15-06-trunk (commercial)
Status: RESOLVED → VERIFIED

Updated

14 years ago
Component: Security: UI → Security: UI
Product: PSM → Core

Updated

10 years ago
Version: psm2.0 → 1.0 Branch
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.