Closed
Bug 81494
Opened 23 years ago
Closed 23 years ago
Duplicate POP limit download size pref: Server, Offline panels-changes in Offline don't take
Categories
(SeaMonkey :: MailNews: Account Configuration, defect, P2)
SeaMonkey
MailNews: Account Configuration
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla0.9.3
People
(Reporter: laurel, Assigned: racham)
Details
(Whiteboard: [nsbeta1+][nsbranch+,pdt+])
Attachments
(1 file)
(deleted),
patch
|
Details | Diff | Splinter Review |
Using may17 commercial trunk build
We have inadvertently wound up with duplicate prefs for the POP limit download
size. It has been residing in the Server panel and when Offline&DiskSpace panel
was added for each account, it appears there as "Do not download message bodies
that are larger than N kb". We only need one instance.
Comments from David Bienvenu via email:
"There should only be one; Laurel is right. Where it goes is debatable. If we
want to be consistent across pop, imap, and news, it should go in the
offline/disk space panel. If we could get rid of the offline disk space panel
for pop, then we should put it in the server settings for pop. I think for ease
of use for pop users, having one less place to look for settings would be good.
But I don't feel strongly about it."
We need to decide where it will reside and remove the other.
Comment 1•23 years ago
|
||
Jennifer, any opinions? I think it would be easier to get rid of the one in the
server panel right now.
moving to 0.9.2, but I'd argue for it for 0.9.1 if a fix were to exist.
Priority: -- → P2
Whiteboard: [nsbeta1+]
Target Milestone: --- → mozilla0.9.2
No strong opinions on this one.
1. Relocate both disk space prefs for POP to server panel and get rid of Offline
& Disk Space for POP. Pro: One less panel.
2. Keep both Disk Space type prefs together on a separate panel. Offline &
Diskspace should really be just "Diskspace" for POP. Pro: parallel location with
IMAP and NEWS accounts so easier for users find location. Will there be
additional items in the future?
Reminder: if we do option #1 and get rid of the Offline panel for POP, we need
to remove the "Offline Settings" link from Account Central for POP accounts.
POP Account Central probably shouldn't have "Offline Settings" anyway, since
there really aren't any offline settings for POP (just Disk Space).
If we're agreed on that, I'll log a separate account central bug?
With these duplicates, checking or unchecking this
option or changing the size limit in the Offline panel does not reflect in the
Server panel. If user closes the Account settings at this point the changes
aren't saved.
The suggested fix is to remove it from the Server panel, if this is done we need
to make sure the Offline panel holds the changes.
FYI..per comment on 5-22-01, I could not find a bug for removing the Offline
Settings from Account Central (which is a separate bug) so I logged one 83594.
Summary: Duplicate POP limit download size pref: Server, Offline panels. → Duplicate POP limit download size pref: Server, Offline panels-changes in Offline don't take
Esther, there is already a bug about the remembering of the pref 82984.
Jennifer,
Just pinging to know if there is a general consent reached on this one in any
of your meetings.
There is a suggestion to get rid of the offline panel altogether for POP and
having the pref in server panel only. But, I think it is better to leave the one
in offline & diskspace panel (I knmow there are offline items in pop panel
today) and remove the one in Server Panel, particularly given the fact that we
are trying reduce the AccountManager window size (bug 82122). So, it will be
better if we can even out the distribution of prefs whereever possible. Also, it
will be nice to see that panel exists for POP also as it offers better user
experience and easier for us to manage the code also (as such there is some
level offline support for POP accounts). If we have to get rid of it
for POP accounts only, I have to start doing special things for POP accounts in
UI to not show offline item.
Anyway, Let me know if this one was discussed later.
bhuvan
Status: NEW → ASSIGNED
Assignee | ||
Comment 11•23 years ago
|
||
Comment 12•23 years ago
|
||
bhuvan, no "official" decision, but so far I think Putterman, you and I agree
that keeping both Disk Space type prefs together on a separate panel is
preferable, so lets go with that.
Assignee | ||
Comment 13•23 years ago
|
||
Time to get reviews and check this in.
Will request diane and Seth for r/sr. Will send a email request also.
bhuvan
Comment 14•23 years ago
|
||
r=dianesun
Comment 15•23 years ago
|
||
sr=sspitzer
Assignee | ||
Comment 16•23 years ago
|
||
Fixed on trunk.
Leaving the bug open until the branch checkin.
Whiteboard: [nsbeta1+]waiting for r= → [nsbeta1+]
Updated•23 years ago
|
Whiteboard: [nsbeta1+] → [nsbeta1+]Have Fix
Verified FIXED (*TRUNK*) :
The setting "Limit message download to X kB per message" is no longer underneath
'Server Settings' in the Account Manager. It remains now (was here before) in
category 'Disk Space', "Do not download messages locally that are larger than X kB"
Windows 2K - 2001-06-29-04
Mac OS 9.1 - 2001-06-29-05
RedHat 7.1 - 2001-06-29-06
Comment 18•23 years ago
|
||
adding vtrunk keyword to indicate the bug has been fixed on the trunk. Bug left
open for tracking checkin to branch (nsbranch) when appropriate.
Once bug has been fixed on the branch also, pls remove vtrunk keyword.
Keywords: vtrunk
Comment 19•23 years ago
|
||
adding nsbranch+. But, can you leave out the dtd change? My understanding is
that even removing strings can mess up localization and having them on the
branch shouldn't cause any harm.
Whiteboard: [nsbeta1+]Have Fix → [nsbeta1+][nsbranch+]Have Fix
Comment 20•23 years ago
|
||
trunkverified
Comment 21•23 years ago
|
||
When the branch is open, please check this into it today.
Whiteboard: [nsbeta1+][nsbranch+]Have Fix → [nsbeta1+][nsbranch+,pdt+]Have Fix
Assignee | ||
Comment 22•23 years ago
|
||
checked in on the branch.
Marking fixed. Removing vtrunk keyword.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Keywords: vtrunk
Resolution: --- → FIXED
Whiteboard: [nsbeta1+][nsbranch+,pdt+]Have Fix → [nsbeta1+][nsbranch+,pdt+]
Reporter | ||
Comment 23•23 years ago
|
||
Verified OK using jul09 commercial 0.9.2 branch builds with win98, mac OS 9.0
and linux rh6.2
Status: RESOLVED → VERIFIED
Reporter | ||
Comment 24•23 years ago
|
||
Added note: the limit download feature isn't working quite right, but there's an
existing bug 74781 on it. Message doesn't auto-refresh when clicking on link to
retrieve the rest of the message.
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•