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)

x86
All
defect

Tracking

(Not tracked)

VERIFIED FIXED
mozilla0.9.4

People

(Reporter: hwaara, Assigned: sspitzer)

References

Details

Attachments

(7 files)

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.
Not sure if it was Diane or Mohan who did this dialog - taking a guess.
Assignee: bienvenu → dianesun
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
Blocks: 85276
No longer blocks: 85276
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 :-)
Whiteboard: Request R&SR
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.
Request R & SR, Bhuvan, Hakan, Seth?
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.
If it's not too much work, I think you should glance at the subscribe tree and try to use that approach (non-css).
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
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
updating summary. this bug has morphed.
Blocks: 73948
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
taking. I plan on fixing this and landing it with the fix for #73865
Assignee: dianesun → sspitzer
Status: ASSIGNED → NEW
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
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.
> 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.
> 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
> 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.
> 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.)
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
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
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: