Closed
Bug 94439
Opened 23 years ago
Closed 22 years ago
need UI for pushed auth (news)
Categories
(SeaMonkey :: MailNews: Account Configuration, defect)
SeaMonkey
MailNews: Account Configuration
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla1.0.1
People
(Reporter: sspitzer, Assigned: neil)
References
()
Details
Attachments
(3 files, 2 obsolete files)
1.68 KB,
patch
|
Details | Diff | Splinter Review | |
18.52 KB,
image/gif
|
Details | |
16.19 KB,
image/gif
|
Details |
need UI for pushed auth (news) spun off of bug #39862 this is low priority, for now users can hack there prefs.
Updated•23 years ago
|
Updated•23 years ago
|
Hardware: PC → All
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•23 years ago
|
||
Is News pushed authentication a priority for the next release?
Not according to the PRD, no. Although some Enterprise customers might find this highly desireable, since pushed auth enables some 'hidden' newsgroups.
*** Bug 140429 has been marked as a duplicate of this bug. ***
*** Bug 144270 has been marked as a duplicate of this bug. ***
Attachment #91407 -
Attachment is obsolete: true
Comment 8•22 years ago
|
||
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".
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.
Assignee | ||
Comment 10•22 years ago
|
||
Assignee | ||
Updated•22 years ago
|
Blocks: 172730
Attachment #91438 -
Attachment is obsolete: true
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•22 years ago
|
||
Adding checkbox for news accounts is fine with me. Please allow Robin to approve wording. Thanks. :-)
Comment 15•22 years ago
|
||
Suggested text for checkbox: "Always request authentication when connecting to this server"
Reporter | ||
Comment 16•22 years ago
|
||
I'll finish this up and drive it in.
Reporter | ||
Comment 17•22 years ago
|
||
checked in, re-assign to neil@parkwaycc.co.uk for credit
Assignee: racham → neil
Status: ASSIGNED → NEW
Reporter | ||
Comment 18•22 years ago
|
||
fixed. thanks for the patch, neil.
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•