45722 - Account Settings: POP Server: UI clean up

Status: VERIFIED WORKSFORME

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

Description

[jglick]

Account Settings dialog, POP Server panel
Minor UI clean up from Mail UE review:

1. Would be nice to have the ability to edit the server name (P4).
2. Group box around server options (P3)
3. Checkbox: "Automatically download any new messages". Child checkbox should be 
disabled if the parent dialog is not checked. (P3)
4. "use secure connection", U should be capitalized. Remove period (.) from end 
of sentence. (P5)
5. "Local" label should be moved a touch to the right over its text box (P5).
6."Delete messages on server when...". Isn't this supposed to be a child of 
"Leave messages on the server" instead of at the same level?
7. Is SSL pref supposed to be here? I seem to recall this item being cut?
8. "Local directory" should say "Local mail directory". (P5)
9. Should the "Local directory" be editable? In 4.x the text field is not 
editable but their is a "Choose" button.

Comment 1

[jglick]

nominate as b3mail

Comment 2

[selmer (gone)]

change b3mail to nsbeta3.

Comment 3

[Alec Flett]

Just checked in some fixes:

#1 is in another bug
#4 is fixed
#5 is fixed
#6 I don't know about this actually. We need to talk to bienvenu or jefft.
#7 nope, so taken out for POP
#8 probably WONTFIX because this string is shared amongst all protocols
#9 Yes, it should be editable, but it needs a choose button.

so all we have left are:
#2
#3
#6 (still open issue)
#9

Comment 4

[selmer (gone)]

Adding mail4 triage nomination keyword.

Comment 5

[lchiang]

+ per mail triage for jennifer's P3 or higher items.

Comment 6

[robinf]

Adding myself to cc: list.

Comment 7

[Alec Flett]

I think this is "good enough" for beta3. Removing nsbeta3 statuses and futuring.
If anyone objects, speak now or forever hold your peace.

Comment 8

[Alec Flett]

massive reassign of account manager bugs -> sspitzer
please feel free to put me back on the CC if you have any questions/comments

Comment 9

[(not reading, please use [e-mail] instead)]

mass re-assign of account manager bugs to racham.

Comment 10

[Håkan Waara]

Mass assigning all racham's UI clean up bugs to me, with permission. Sorry for
the spam.

Comment 11

[Håkan Waara]

As far as I can see all bugs are fixed by now.

Suggesting FIXED/WORKSFORME.

Comment 12

[Ninoschka Baca]

Marking Worksforme. 

jglick: It would be great if the Account Setup UI spec could be updated to 
reflect the order of entries in the POP Server panel that's in the product now.
I'm sure there will be additional entries but for documentation purposes I'm 
listing what's present right now: 

- Server Type
- Server Name
- User Name
- Port
- Log onto this account at startup of Mail
- Check for new messages every # minutes
  - Automatically download any new messages
- Leave messages on server
  - Delete messages on server when they are deleted locally
- Limit message download to # kB per message
- Empty Trash on Exit
- Local Directory

Comment 13

[Ninoschka Baca]

Worksforme.

Comment 14

[Ninoschka Baca]

Verified Worksforme. 
All three platforms checked.

Comment 15

[jglick]

Ninoschka, sorry for the delay. Account Settings spec is not updated and 
reflects your summary.
http://www.mozilla.org/mailnews/specs/accounts/

Comment 16

[jglick]

Oops, NOW updated.

Comment 17

[Ninoschka Baca]

Jen, In the spec "Limit message download to # kB per message" appears after
"Empty Trash on Exit". In the product it appears before "Empty Trash on Exit".
Should the position be changed in the product or is it a mistake in the spec?

Comment 18

[jglick]

I don't think the order is real important as long as the items are there. Also, 
"Limit message download to # kB per message" could possibly be moved to an 
"Offline and Storage" panel per the Offline discussions.