Mailing List needs a different icon in results tree

VERIFIED FIXED in mozilla0.9.8

Status

SeaMonkey
MailNews: Address Book & Contacts
P3
normal
VERIFIED FIXED
18 years ago
13 years ago

People

(Reporter: Suresh, Assigned: (not reading, please use seth@sspitzer.org instead))

Tracking

({polish})

Trunk
mozilla0.9.8
polish
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments)

(Reporter)

Description

18 years ago
Select a AB in the left pane. In the right pane, you will see the AB card and AB
lists are displayed in the same icon. Mailing lists needs a different icon.

Build and platform:
Today's commercial build on all platforms.
(Reporter)

Comment 1

18 years ago
Changing QA Contact.
fyi: There is a similar kinda bug - 26112. Not sure whether it is a same issue :)
QA Contact: esther → suresh

Updated

18 years ago
Assignee: putterman → hewitt

Comment 2

18 years ago
reassigning to hewitt. I think Joe had mentioned turning these back on and the 
possibility of new icons.

Comment 3

18 years ago
We do indeed have distinct icons in hand for mailing lists on both modern and
classic.
Status: NEW → ASSIGNED

Updated

18 years ago
QA Contact: suresh → pmock

Comment 4

17 years ago
QA-assign-to fenella.
QA Contact: pmock → fenella

Comment 5

17 years ago
Created attachment 27960 [details]
The mailing list icon used in left pane but not in rt pane

Updated

17 years ago
Keywords: nsCatFood, polish

Updated

17 years ago
QA Contact: fenella → nbaca

Comment 6

17 years ago
Mass reassigning some of my theme bugs to Shuehan.
Assignee: hewitt → shliang
Status: ASSIGNED → NEW

Updated

17 years ago
Status: NEW → ASSIGNED
Target Milestone: --- → Future

Updated

17 years ago
Keywords: nsbeta1
I've got the fix for this in my tree.  it's not a CSS issue.

I'll attach a screen shot.
Assignee: shliang → sspitzer
Status: ASSIGNED → NEW
Status: NEW → ASSIGNED
Summary: Mailing List needs a different icon → Mailing List needs a different icon in results tree
Target Milestone: Future → mozilla0.9.6
Depends on: 105835
patch coming, it will include this fix, and a fix for #105835, and some
performance  / cleanup changes. 
Created attachment 55283 [details] [diff] [review]
initial patch, include fix for #105835, clean up, performance fixes.

Updated

17 years ago
Keywords: nsbeta1 → nsbeta1+

Comment 11

17 years ago
Comment on attachment 55283 [details] [diff] [review]
initial patch, include fix for #105835, clean up, performance fixes.

>+treeitem[empty="true"] > treerow > treecell > .tree-cell-twisty {               
>+  visibility: hidden;
>+}    
>+

Is this hack still valid?  We use outliners everywhere, 
so I doubt it is.

>+	if (name.Length() > 0) {
>+	  rv = createNode(name.get(), target);

Use if(!name.IsEmpty()) instead.

There are a bunch of tabs being added also...

Really nice cleanup.
pushing this off.  

this fix will have to be redone, since we're switching to outliner.

>+treeitem[empty="true"] > treerow > treecell > .tree-cell-twisty 
{               
>+  visibility: hidden;
>+}    
>+

yes, it's needed for classic and modern.  there are trees in both products, 
account manager tree, pref tree, etc.

Target Milestone: mozilla0.9.6 → mozilla0.9.9
fixed for both modern and classic as part of AB_OUTLINER_BRANCH landing.
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
Target Milestone: mozilla0.9.9 → mozilla0.9.8

Comment 14

16 years ago
Trunk build 2002-01-28-03: WinMe
Trunk build 2002-01-28-08: Linux RH 7.1
Trunk build 2002-01-28-08: Mac 10.1
Verified Fixed, using Modern and Classic.
Status: RESOLVED → VERIFIED

Updated

15 years ago
Depends on: 73868
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.