Closed Bug 46450 Opened 24 years ago Closed 24 years ago

UI:Account Wizard:Summary Panel: show summary of all settings

Categories

(SeaMonkey :: MailNews: Account Configuration, defect, P2)

x86
Windows 98
defect

Tracking

(Not tracked)

VERIFIED FIXED
mozilla0.8

People

(Reporter: jglick, Assigned: racham)

References

Details

(Keywords: polish, Whiteboard: [nsbeta1+])

Attachments

(14 files)

Account Wizard: Summary Panel(Congraduations panel). Show a summary of all 
the account settings the user has selected.

Priority 5. Polish.
Keywords: nsbeta3, polish
Target Milestone: --- → M18
QA Contact: lchiang → nbaca
Mail triage is marking nsbeta3-
Whiteboard: [nsbeta3-]
Target Milestone: M18 → Future
massive reassign of account manager bugs -> sspitzer
please feel free to put me back on the CC if you have any questions/comments
Assignee: alecf → sspitzer
I checked the spec and I don't see specifically what information should be
displayed for the following:
- POP/IMAP
- Netscape Webmail
- AOL
- News

For News I would like to see:
- Username
- Email address
- News Server Name
Adding mail3 as a keyword. Giving the user a summary of all the information is
good. This allows them to go back if a mistake was made and could be a
reinforcement that they entered the information correctly.
Keywords: mail3
about news, we should not be showing the username.
we have a related bug about showing the username for news in the account manager.

the problem is the username (and password) we actually use is stored in the
password manager.  news usernames can vary per newsgroup on a server, so we
store it on a group by group basis (and for the server.)


Thanks for clarifying the news issue.
markings nsbeta1+ and moving to mozilla0.8.  reassigning to racham.
Assignee: sspitzer → racham
Keywords: nsbeta3nsbeta1
Priority: P3 → P2
Whiteboard: [nsbeta3-] → [nsbeta1+]
Target Milestone: Future → mozilla0.8
I have added code to enable the display of most of the information we collect
from the user as a Summary on the last screen of Account Wizard.

I will post the patch and also attach 2 snapshots taken after creating a mail
and news accounts respectively.

As we are using trying to hide the fields where we do not have data or the
fields not relavant in the context, on the summary panel of mail/news account
you will find a blank line(s) separating some of these items. Snapshots included
hopefully are in reasonable shape for presenting the info to the user. Anyway,
input and ideas are welcome here.

Adding seth, bienvenu and ninoschka to the cc list. Jennifer has reported this bug.
Status: NEW → ASSIGNED
Keywords: patch
comments:

notice the "..." in your screen shots.  My guess is the text is getting cropped,
so layout puts the "..." in there.

the problem is with your new entities in aw-done.dtd

you have:
+<!ENTITY serverTypePrefix.label "Incoming Server Type:      ">
+<!ENTITY serverNamePrefix.label "Incoming Server Name:      ">
+<!ENTITY smtpServerNamePrefix.label "SMTP Server Name:">
+<!ENTITY newsServerNamePrefix.label "News Server Name:      ">
you should have:
+<!ENTITY serverTypePrefix.label "Incoming Server Type:">
+<!ENTITY serverNamePrefix.label "Incoming Server Name:">
+<!ENTITY smtpServerNamePrefix.label "SMTP Server Name:">
+<!ENTITY newsServerNamePrefix.label "News Server Name:">

it looks like the xul is already written to line things up correctly.

also, what do pop, webmail and AOL accounts look like?  
can you post screen shots for your local commercial build?

I'm guessing that imap, webmail and AOL accounts will say "Incoming Server Type:
imap" and pop will have "Incoming Server Type: pop3"

I think we should have:

IMAP (instead of imap)
POP3 (instead of pop3)
AOL Mail (instead of imap)
Netcenter WebMail (instead of imap)

once you fix the "..." problem, r=sspitzer. 

then, open a new bug on allowing for a pretty name for the type.  I've got some
ideas on how to do that I'll discuss with bhuvan.
*** Bug 37602 has been marked as a duplicate of this bug. ***
>I think we should have:

>IMAP (instead of imap)
>POP3 (instead of pop3)
>AOL Mail (instead of imap)
>Netcenter WebMail (instead of imap)

I agree.

I would expect POP3 to look almost identical to IMAP (except for Incoming 
Server Type of course).  AOL and Webmail would look similar to Mail accounts 
except Incoming and SMTP Server Names are not needed since the user didn't enter 
these themselves (and hence may confuse them).  Incoming Server Type: "AOL Mail 
- IMAP" or "Netcenter Mail - IMAP".  (" - IMAP" can be left off if folks think 
that is better).
I've logged #63775 to track the other issue.  (cleaning up the summary for AOL
accounts and Netcenter WebMail accounts.)

With some more polish on display after discovering the ability to collapse
unwanted rows and manage between distance columns, I have new screenshots and
the patch to post.

Starting with screenshots..please ignore previous postings.
Attached image News Account screenshot
Seth, Scott/Bienvenu,

Please go through the patches posted and provide r= and sr=.
They are slightly modified frommy previous postings. They are simple and
shouldn't consume much time.

thanks
bhuvan
r=sspitzer

1)  I wouldn't check in your ns tree change without checking the spec.  that
will give all new aol accounts the pretty name of "AOL Mail".  double check with
kmurray, nbaca, jglick.  (I know we do that for "Netcenter Web Mail", but AOL
Mail might be different on purpose.)

2) no need to log that other bug, it looks like I did it already: #63775
As far as I know the pretty names are:

- AOL Mail
- Netscape Webmail
Adding Kevin to the cc list.
so we are supposed to show "AOL Mail" in the folder pane by default?

if that is the case, your change to the ns tree is correct and will fix that
problem.
+        if (accountName == "")
and similar
=> if (!accountName)
+    if (userName == "" && email != "") {
=> if (userName && !email) {

!     if (!value || value =="") {
!     if (!value) {

Also, disabled items shouldn't be defaults (Next is disabled and default in one 
of the pictures) if you don't want to fix that here, could you file a bug for 
it?
The screen shots look excellent. Two things:
1. Regarding nbaca's comment on 1/9 at 16:26, the "pretty name" for webmail is
"Netscape WebMail" (note the capitalized M), and is wrong in the folder pane as
well (I know, file a bug)
2. On the AOL summary screen(id 22184), we need to slim down the info. "User
Name" should be "Screen Name" and we should remove the following lines/info:
"Incoming Server Name," "Incoming Server Type," and "SMTP Server Name."  Why?
You know why. Because AOL users won't care about that info.
Kevin,

Regd the first issue you brought up..it's pretty easy to fix 'Webmail' to
'WebMail'. I can do that along with this fix.

Regd your second issue, I would like tackle that in a separate bug which need to
be filed in bugscape (not in bugzilla).

Question posed by Seth about 'AOL Mail' still remains unanswered about 'Aol
Mail' as the pretty Account name on Aol screenshot. I guess you implied that it
is OK to have prettyname 'AOL Mail' for aol accounts. But I want to confirm on
that one explicitely so that we don't overwrite some existing behavior
accidentally..Please comment on that.
does this change make it so that all aol mail accounts show up in the folder
pane as just "AOL Mail" and not Mail for joe@aoo.com? That would mean that if I
had multiple AOL accounts, they would all show up as just AOL Mail and I could
not tell the difference between the accounts. That would be very confusing for me.
I think Seth was just asking about the default, which should be rename-able.
Screen shots look good. Some comments.

News summary screen: Do we want to show "User Name"?  I remember some comment 
about not showing it. Maybe a comment from Seth?

Agree with Kevin about AOL Mail and WebMail summary screens. They shouldn't show 
all that info. Only the info that the user actually entered or needs to know.

AOL: Email address, Account Name.  (Screen Name - the wizard currently asks 
users to enter their email address, but they are really only entering their 
screen name since the static text '@aol.com' is shown.  Could the wizard ask 
users to enter their screen name instead since that is more accurate?  If so, 
then also show "Screen Name" on the summary page.)

WebMail: Email Address, Account Name.  ("Screen Name" or "User Name"?  same 
comments as above, but does netcenter call it a User Name or a screen name?  
The wizard should also be changed to ask for User/Screen Name instead of email 
address since that is what the text field is asking for).
We need to change the wizard to ask for "Screen Name" as well. Regarding
WebMail, they use "User Name."
I updated the Account Wizard screens of the Account Settings spec. It doesn't 
show up yet, but should be available soon.

http://www.mozilla.org/mailnews/specs/accounts/

Note: red text for aol and webmail screens is what I would like to see the 
wording changed to.
yes, jglick is right.  don't show the username for news.

but feel free to log that as a bug and fix it later, along with the other polish
items.
Jen, did you mean to change the wording on the WebMail screen back to Netcenter?
Kevin, what should it say?  "Enter you WebMail user name?".  Let me know what 
the correct wording is and i'll update the spec.  Thanks.
Good question - since it's no longer called "Netcenter" we should probably call 
it "Netscape.com User Name." 
Here is an updated patch after our discussions.

* News account summary panel will not have username.

For all AOL and WebMail issues, please refer to bugscape bug 3674.

New patch coming up..
Attached image New news screenshot
ok racham, what's the *real* bugscape number? ;-)
It is 3674...
Here you go : http://bugscape.netscape.com/show_bug.cgi?id=3674
sr=bienvenu. However, I still have to say that just showing AOL Mail in the
folder pane without indicating the user/screen name is going to be very
confusing for AOL users with multiple e-mail names. This is a very common useage
- a family shares an AOL account, and has separate e-mail accounts for each
member. We would be requiring them to have separate netscape profiles in order
to distinguish between their aol accounts (because they would all just show up
as "AOL Mail" in the folder pane), and I believe that setting up separate
profiles is a very difficult thing to require AOL users to do. I thought we were
trying to get away from setting up separate profiles.
bienvenu, racham, and jglick: 

instead of "AOL Mail" for every aol account, how about "AOL Mail for
<screenname>.

we should allow racham to land his patch (to get it out of his tree) and then
log a new bug to fix that issue.
I am going to do 'AOL mail for <screenname>'. That is the plan. I will file a
new bug. Just didn't bring that up here. I don't want to delay this furher.
Thanks for review and superreviews.
I think "AOL Mail for <screenname>" is good, as well as "WebMail for
<username>". We had talked about "AOL Mail for <emailaddress> at the issues 
meeting, but I thought that was too long and actually did what Seth suggested in 
the spec.

http://www.mozilla.org/mailnews/specs/accounts/images/WizardAol6.gif
We will go with just the screenname/username (ISP's user's email id).

I already open a bug (http://bugscape.netscape.com/show_bug.cgi?id=3709) on this
issue. thanks.
Fixed.

Thanks for reviews/super reviews && suggestions.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Build 2001-02-19-08: NT4, Linux RH 6.2, Mac 9.04
Verified Fixed.
Status: RESOLVED → VERIFIED
These are samples of what was displayed.

POP/IMAP Summary:
Account Name:         qatest20@netscape.com
User Name:            qatest20
Email Address:        qatest20@netscape.com
Incoming Server Name: nsmail-12
Incoming Server Type: POP3/IMAP
SMTP Server Name:     nsmail-12

News Summary:
Account Name:     zia
Email Address:    qatest20@netscape.com
News Server Name: zia
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: