94439 - need UI for pushed auth (news)

Status: RESOLVED FIXED

(SeaMonkey :: MailNews: Account Configuration, defect)

Description

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

need UI for pushed auth (news)

spun off of bug #39862

this is low priority, for now users can hack there prefs.

Comment 1

[lah]

The Outgoing server (SMTP) dialog have a 'Use name and 
pasword' checkbox and a 'User Name' entry (Why a capital 
'N' BTW ;-)

Minimal change is a refrasing to 'Force use of 
name and password' for the checkbox.

The username entry is not needed, but is good for 
consistency. But the checkbox alone might fit 
cleaner in the curent dialog thou :-/ (and in 
the needed funktionality). There tree rows of 
chekboxes, one more (under 'use SSL') would look 
werry clean. 

Lots of users, and nonhacker users, need this feature 
so a GUI is important, if not absolutly needed.

Wher is this gui files... seams easy ;-) Any pointer 
to docs (and preferably the right file ;-)

Any preemaid design desission one need to know before 
start hacking? This is a little thing to do so I don't 
feel motivated to read a pile of docs, only the most 
important ;-)

Comment 2

[Ninoschka Baca]

Is News pushed authentication a priority for the next release?

Comment 3

[[e-mail] (gone - use [e-mail] instead)]

Not according to the PRD, no.  Although some Enterprise customers might find
this highly desireable, since pushed auth enables some 'hidden' newsgroups.

Comment 4

[[e-mail] (gone - use [e-mail] instead)]

*** Bug 140429 has been marked as a duplicate of this bug. ***

Comment 5

[[e-mail] (gone - use [e-mail] instead)]

*** Bug 144270 has been marked as a duplicate of this bug. ***

Comment 6

[[e-mail] (gone - use [e-mail] instead)]

Attachment: Preliminary patch, this currently doesn't work and has bad alignment.

Comment 7

[[e-mail] (gone - use [e-mail] instead)]

Attachment: Yet another patch, fixes UI issues, doesn't work either.

Comment 8

[Aleksey Nogin]

IMHO the "Always ask me for my username and password" string is pretty wrong. 
First, it suggest that this is a preference for overriding the password wallet.
Second, it does not say anything about actually sending this auth info to the
server.

I would suggest something along the lines of "Always authenticate me when
connecting to this server".

Comment 9

[[e-mail] (gone - use [e-mail] instead)]

Once the patch itself works, we can fix the wording.  I won't post any more
updates until I've tested and verified a patch that actually works.

Comment 10

[[e-mail]]

Attachment: Proposed patch

Comment 11

[[e-mail] (gone - use [e-mail] instead)]

Attachment: Screenshot of Modern

Comment 12

[[e-mail] (gone - use [e-mail] instead)]

Attachment: Screenshot of Classic

Comment 13

[[e-mail] (gone - use [e-mail] instead)]

I've attached two screenshots that show the Account Manager's height when this
pref is added.  It looks fine in both Modern and Classic. 

I've also personally tested the patch and found it to work fine.  Remember that
the functionality for this was verified back in bug 39862.

The next step is to make sure the wording is fine with Jennifer and Robin, then
seek reviews.

Comment 14

[jglick]

Adding checkbox for news accounts is fine with me. Please allow Robin to approve 
wording. Thanks. :-)

Comment 15

[robinf]

Suggested text for checkbox: "Always request authentication when connecting to
this server"

Comment 16

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

I'll finish this up and drive it in.

Comment 17

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

checked in, re-assign to [e-mail] for credit

Comment 18

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

fixed.  thanks for the patch, neil.