Closed
Bug 85088
Opened 23 years ago
Closed 23 years ago
"Select Offline" dialog needs to be switched to use the outliner
Categories
(SeaMonkey :: MailNews: Backend, defect, P2)
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla0.9.4
People
(Reporter: hwaara, Assigned: sspitzer)
References
Details
Attachments
(7 files)
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
image/gif
|
Details | |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
image/gif
|
Details | |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
image/gif
|
Details |
Because the "Select Offline" dialog uses the same rules as the folderpane does,
switching the folderpane over to outliner (and thus changing the rules to be
outliner-specific) will break this tree's icons.
We need to make it so this dialog can use its icons regardless of folderpane's
rules... Like the subscribe dialog does it.
I guess this should be high priority, but this is really your call.
Comment 1•23 years ago
|
||
Not sure if it was Diane or Mohan who did this dialog - taking a guess.
Assignee: bienvenu → dianesun
Assignee | ||
Comment 2•23 years ago
|
||
short term: we'll need a new .css file for this dialog. probably copy the old
folerpane.css to offline.css, and reference that from the "Select Offline"
dialog.
long term: like all other multiple column tree widgets, we'll need to switch
to the outliner.
Status: NEW → ASSIGNED
Priority: -- → P2
Target Milestone: --- → mozilla0.9.3
Comment 3•23 years ago
|
||
Perhaps as a workaround you can use the folderMenuItem rules? They are almost
the same as the folderTreeItem rules except that they apply to all tree columns
so that the checkboxes look odd :-)
Reporter | ||
Comment 5•23 years ago
|
||
I take it you just copied folderPane.css and made it offline.css?
There are many rules there that the offline command doesn't need IIRC.
Also, this doesn't have to land before the folder_outliner branch.
In the modern theme, it is not completed copied since a lot of items we don't
need in the offline.css.
Reporter | ||
Comment 8•23 years ago
|
||
I don't think this fix is the best possible fix. You shouldn't have to use CSS
at all to fix this bug.
Maybe look at how the subscribe tree does it?
The Select Offline dialog doesn't need to use nearly as many CSS rules as the
folderpane does.
It is true we don't use many CSS rules in the select offline dialog. Any reason
not to use CSS at all, or we should use other approach, please advise.
Reporter | ||
Comment 10•23 years ago
|
||
If it's not too much work, I think you should glance at the subscribe tree and
try to use that approach (non-css).
Assignee | ||
Comment 11•23 years ago
|
||
diane, hold of on this bug.
we need to switch this dialog from using a tree to using an outliner.
I'm working with hwaara and jan to land the folder pane outliner changes, and
after that, we'll work on converting this dialog to the outliner.
if we do that right, this dialog can continue to use folderpane.css.
I'll investigate how much work it will be to do that, and get back to you.
it shouldn't be much.
Depends on: 73865
Assignee | ||
Updated•23 years ago
|
Summary: "Select Offline" dialog should not inherit folderpane.css rules → "Select Offline" dialog needs to be switched to use the outliner
Whiteboard: Request R&SR
Assignee | ||
Comment 12•23 years ago
|
||
updating summary. this bug has morphed.
Comment 13•23 years ago
|
||
Doesn't look like this is getting fixed before the freeze tonight. Folder pane
outliner has a few kinks in it, as well. Pushing out a milestone. Please
correct if I'm mistaken.
Target Milestone: mozilla0.9.3 → mozilla0.9.4
Assignee | ||
Comment 14•23 years ago
|
||
taking. I plan on fixing this and landing it with the fix for #73865
Assignee: dianesun → sspitzer
Status: ASSIGNED → NEW
Assignee | ||
Comment 15•23 years ago
|
||
ok, I've got this started.
I'll attach a patch.
I've got some issues I still need to figure out before it is complete.
Status: NEW → ASSIGNED
Assignee | ||
Comment 16•23 years ago
|
||
Assignee | ||
Comment 17•23 years ago
|
||
Assignee | ||
Comment 18•23 years ago
|
||
that patch won't work unless you've got the lastest patch to #73865.
here are the things I need to finish before this is ready to land:
1) only show those accounts that support offline (currently, pop is showing up.)
2) column header titles aren't showing up
3) checkbox images for the "synch" column
4) checkboxes for folders only (not servers)
5) I'm performing expand on the imap servers, but it's not prompting me for
login.
6) toggling synch state works, but not showing up in the outliner until I force
it.
Assignee | ||
Comment 19•23 years ago
|
||
> 1) only show those accounts that support offline
> (currently, pop is showing up.)
> 2) column header titles aren't showing up
working on a fix to #2 (thanks to jan for the help).
jan's also got some ideas about #1, we're talking over aim.
Assignee | ||
Comment 20•23 years ago
|
||
Assignee | ||
Comment 21•23 years ago
|
||
Assignee | ||
Comment 22•23 years ago
|
||
> 1) only show those accounts that support offline
> (currently, pop is showing up.)
> 2) column header titles aren't showing up
thanks to some help from jan, both of those are fixed.
now on to items 3-6
Assignee | ||
Comment 23•23 years ago
|
||
> 3) checkbox images for the "synch" column
> 4) checkboxes for folders only (not servers)
ok, I got 3 and 4 working. here comes a new diff and new screen shot.
Assignee | ||
Comment 24•23 years ago
|
||
Assignee | ||
Comment 25•23 years ago
|
||
Assignee | ||
Comment 26•23 years ago
|
||
> 6) toggling synch state works, but not showing up in the outliner until I
force it.
I've got this fixed. the patch will be included in the patch for #73865
two remaining problems:
5) I'm performing expand on the imap servers, but it's not prompting me for
login.
7) not always painting check / dots (sometimes I have to force a repaint.)
Assignee | ||
Comment 27•23 years ago
|
||
fixed. there are some issues and open bugs to fix, but the basics work.
this needed to be done since folderPane.css got rewritten, and this is really a
folder pane.
I'll go log any new bugs that I know about, and cc dianesun and gchan.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Comment 28•23 years ago
|
||
Commercial builds
2001082703 - win nt 4.0
2001082708 - linux 2.2, mac 9.0.4
Verified on both modern/classic themes.
Other than bugs already filed (bug 95377 , bug 95378) looks
ok to me.
Marking as verified.
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
•