21533 - Need dialog with two password fields

Status: VERIFIED DUPLICATE of bug 19086

(SeaMonkey :: UI Design, defect, P3)

Description

[Stephen P. Morse]

I thought there always was an open bug report on this but I couldn't find it.
So I'm opening up this report.

Per the wallet UI meeting that was held several months ago (see bug 13022), the
need for a dialog with two password fields was discovered.  Besides the password
fields, the dialog must have a text field, an OK button, and a CANCEL button.

Comment 1

[Matthew T (active 1999-2002)]

Is this widely used enough to need a generic dialog? I mean, what else is going
to use it, besides the Wallet (bug 21324)?

[Presumably, bug 21324 is dependent on this bug ...?]

Comment 2

[Stephen P. Morse]

davidm can answer that question better than I can but I'll tell you what I
believe he is going to do.  He'll provide a call to a generic dialog with
parameters that specify the number of text fields, check boxes, password fields,
etc.  So what this bug report is asking for is the ability of the generic dialog
routine to put up a dialog with two password fields.

The other alternative is for each developer to maintain his own customized
dialog.  But this has pitfalls that I've already run into.  I did so for yes/no
dialogs and davidm was maintaining the ok/cancel dialog.  Then he made a major
change to his dialogs because of threading problems and the result was that
the ok/cancel dialog worked just fine but the yes/no dialog didn't.  So a
centralization of dialogs is needed in order to keep them all in synch.

Comment 3

[davidm]

There are a couple issues here. 1) wallet really doesn't have enough information
to put up dialogs. 2) in order to allow embedding all dialogs should go through a
common interface ( or 2). My plan is to add a couple of more routines to
nsIPrompt ( which perhaps should actually be split into 2 interfaces 1) with the
simple stuff needed by the DOM and 1 with our fancy dialogs), a Select Dialog (
done ), 2 Passwords, and a 3 button dialog which takes in user specified text for
the buttons title and command keys.

Comment 4

[davidm]

*** This bug has been marked as a duplicate of 19086 ***

Comment 5

[Eli Goldberg]

I'm rubber-stamping this as duplicate, as I fear the newsgroup & bug report
Stored Logins dialog discourse has exceeded my memory capacity. ;)

morse or mpt, please re-open with your $.02 if you believe this should be kept
open. Thanks!

Comment 6

[Eli Goldberg]

Changing component to XP Apps. (HTML Dialogs is going away.)