Closed
Bug 82454
Opened 23 years ago
Closed 23 years ago
properly enable / disable "File | Offline | Get [Selected / Flagged] Messages"
Categories
(SeaMonkey :: MailNews: Backend, defect, P2)
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla0.9.4
People
(Reporter: sspitzer, Assigned: mohanb)
References
Details
(Whiteboard: ready for checkin, verified on trunk)
Attachments
(2 files)
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review |
properly enable / disable "File | Offline | Get [Selected / Flagged] Messages"
spun off of #79104
we'd do this in the oncreate for that menupopup.
I'm not sure if there is an efficient way to know if a folder has any flagged
messages. that could be why 4.x always had get flagged messages enabled. (just
a guess)
Updated•23 years ago
|
Summary: properly enable / disable "File | Offline | Get [Selected / Flagged] Messages" → properly enable / disable "File | Offline | Get [Selected / Flagged] Messages"
Assignee | ||
Comment 1•23 years ago
|
||
assigning myself.
Assignee | ||
Updated•23 years ago
|
Priority: -- → P2
Target Milestone: --- → mozilla0.9.2
Assignee | ||
Comment 3•23 years ago
|
||
bienvenu,
1. we need to check whether a folder has flagged/selected messages or not
2. we need a folder property/function for current folder (may be a property of
nsIMsgFolder)
right ?
Comment 4•23 years ago
|
||
We're going to do what 4.x does - enable/disable get selected based on
selection, which you already have access to, and always enable Get Flagged.
Assignee | ||
Comment 5•23 years ago
|
||
Comment 6•23 years ago
|
||
does this patch check if any messages are selected before enabling/disabling
GetSelected Messages? I don't see that it does. The flagged part looks OK.
Assignee | ||
Comment 8•23 years ago
|
||
Comment 9•23 years ago
|
||
r=bienvenu for that second patch.
Assignee | ||
Updated•23 years ago
|
Whiteboard: [PDT+] → [PDT+] requesting r & sr
Assignee | ||
Updated•23 years ago
|
Whiteboard: [PDT+] requesting r & sr → [PDT+] requesting sr
Assignee | ||
Comment 10•23 years ago
|
||
cc'ing sspitzer for super review;
Comment 11•23 years ago
|
||
Unload sspitzer and racham. r=hwaara. bienvenu's r= is sufficent for sr=, so you
only need a= now.
Assignee | ||
Updated•23 years ago
|
Whiteboard: [PDT+] requesting sr → [PDT+] requesting a=
Assignee | ||
Updated•23 years ago
|
Whiteboard: [PDT+] requesting a= → [PDT+] requesting a= Planning for checkin by FRI : 06/22
Assignee | ||
Updated•23 years ago
|
Whiteboard: [PDT+] requesting a= Planning for checkin by FRI : 06/22 → [PDT+] requesting a=
Comment 12•23 years ago
|
||
Mohan, did you email drivers@mozilla.org and CC the pdt? I don't see a request
for a= for this bug in my email (although it may be somewhere I didn't expect
it.) If you did not send this email, the a= might take a while to appear...
Comment 13•23 years ago
|
||
a= asa@mozilla.org for checkin to 0.9.2.
(on behalf of drivers)
Assignee | ||
Updated•23 years ago
|
Whiteboard: [PDT+] requesting a= → [PDT+] ready for checkin
Updated•23 years ago
|
Whiteboard: [PDT+] ready for checkin → [PDT+] ready for checkin, critical for 0.9.2
Comment 14•23 years ago
|
||
This is a 0.9.2 bug. What's the status on getting this checked in?
Updated•23 years ago
|
Whiteboard: [PDT+] ready for checkin, critical for 0.9.2 → [PDT+] ready for checkin
Comment 15•23 years ago
|
||
pushing out. 0.9.2 is done. (querying for this string will get you the list of
the 0.9.2 bugs I moved to 0.9.3)
Target Milestone: mozilla0.9.2 → mozilla0.9.3
Comment 16•23 years ago
|
||
Removing PDT+, marking nsbranch; should be checked into the 0.9.3 trunk ASAP,
tested & verified, then submitted for limbo builds.
Keywords: nsBranch
Whiteboard: [PDT+] ready for checkin → ready for checkin
Comment 17•23 years ago
|
||
Fix checked in on behalf on mohanb.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Comment 18•23 years ago
|
||
Adding vtrunk; bugs need to be verified in the trunk for eventual limbo checkins.
Status: RESOLVED → ASSIGNED
Keywords: vtrunk
Comment 19•23 years ago
|
||
Clearing resolution, nsBranch bugs should be closed only after checkins into
limbo builds.
Resolution: FIXED → ---
Comment 20•23 years ago
|
||
Commercial builds
2001-07-05-09-trunk/ win nt 4.0
2001-07-05-08-trunk/ linux 2.2, red hat 7.0
2001-07-05-08-trunk/ mac os 9.0.4
Verified the following:
* File|Offline|Get Flagged Messages
-if online, it's always available whether a message is flagged or not
-if message(s) is/are flagged, then they are downloaded
* File|Offline|Get Selected Messages
-Enabled
* if messages are selected and messages that are selected
do get downloaded
-Disabled
* if no messages are selected
* Get Flagged/Selected Messages are disabled if you are offline
Verified on trunk.
Removing Keyword Vtrunk.
adding 'verified on trunk' to status whiteboard.
Leaving status as Assigned per instructions from Jussi.
Keywords: vtrunk
Whiteboard: ready for checkin → ready for checkin, verified on trunk
Assignee | ||
Comment 22•23 years ago
|
||
changing the resolution to fixed,
since it is checked into the trunk;
Status: ASSIGNED → RESOLVED
Closed: 23 years ago → 23 years ago
Resolution: --- → FIXED
Comment 23•23 years ago
|
||
Commercial trunk builds
2001082803 - win nt 4.0
2001082808 - linux 2.2, mac 9.0.4
Verified the following:
* File|Offline|Get Flagged Messages
-if online, it's always available whether a message is flagged or not
-if message(s) is/are flagged, then they are downloaded
* File|Offline|Get Selected Messages
-Enabled
* if messages are selected. The messages that are selected
do get downloaded
-Disabled
* if no messages are selected
* Get Flagged/Selected Messages are disabled if you are offline
Status: RESOLVED → VERIFIED
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•