69684 - subscribe dialog slow to update check marks

Status: VERIFIED FIXED

(SeaMonkey :: MailNews: Message Display, defect, P2)

Description

[Asa Dotzler [:asa]]

sorry for the horrible summary but my brain is dead this morning.  

To reproduce open the subscribe dialog for a news server.  Then expand the list
of groups and click on one of the dots to subscribe to a group.  Normally I'd
expect that dot to be replaced with a little blue checkmark  indicating that I
had subscribed to that group. In recent builds (noticed it today and yesterday
but probably been around longer) when I click that dot I don't see the checkmark
until I otherwise manipulate the tree like selecting a row. I see the same
problem in reverse. If you try to unsubscribe by clicking one of the checkmarks
that change isn't reflected visually until I make some other change to the tree.
 This missing visual indication does not seem to affect that actual subscribed
state.

didn't see this reported in the short list of open subscribe bugs but this is
probably a dupe.

Comment 1

[[e-mail] (gone - use [e-mail] instead)]

Thanks for reporting this, Asa.  It's not in a bug I own, so not a DUP so far. 
I can't imagine what has landed top change this, but afaik, nothing has been
checked in to either front end mail code or news code to affect this, however
I'm seeing this on all build platforms.

Comment 2

[laurel]

This behavior is often so bad I give up ... If I didn't know this bug existed, I
would think subscribe/unsubscribe in the subscribe ui is just not working. I'm
nominating. 

Comment 3

[[e-mail] (gone - use [e-mail] instead)]

I noticed that this happened around the same time that the "2nd time to launch
Subscribe window" bug was filed by Seth.  Seth, what API's do we in news rely on
for this dialog?  Could this be attributed to some XUL/RDF stuff?

Comment 4

[(not reading, please use [e-mail] instead)]

do you see it for news.mozilla.org, on the modern skin?

I don't see that on the branch I'm working on, so something must have regressed
on the trunk.

I'll investigate when I get back to a trunk build.

Comment 5

[Asa Dotzler [:asa]]

I see it in the classic skin on news.mozilla.org.  I have not tested the modern
theme.

Comment 6

[[e-mail] (gone - use [e-mail] instead)]

Yes, all servers, with the modern skin.

Comment 7

[(not reading, please use [e-mail] instead)]

bug reported on 2-21, I've been on the branch since 2-08.

that will help me narrow it down.

Comment 8

[[e-mail] (gone - use [e-mail] instead)]

At first I thought this occured around the time you filed
http://bugzilla.mozilla.org/show_bug.cgi?id=61537, but I don't think so, that
was way back in November.

Comment 9

[(not reading, please use [e-mail] instead)]

yikes, I've got a trunk build.  damn, that is slow.

it will be interesting to see what could have caused this regression.

Comment 10

[[e-mail] (gone - use [e-mail] instead)]

Both on linux and win2000, this seems noticeably better using builds 2001022805
(win32)and build 2001022808 (linux).  Anybody else agree?

Comment 11

[[e-mail] (gone - use [e-mail] instead)]

Hmm..that was only for the first time after launching seamonkey.  After
restarting mozilla, the checkmarks are slow again.

Comment 12

[scottputterman]

marking nsbeta1+, it's pretty slow.

Comment 13

[[e-mail] (gone - use [e-mail] instead)]

*** Bug 70592 has been marked as a duplicate of this bug. ***

Comment 14

[(not reading, please use [e-mail] instead)]

this is the cause of many of the subscribe problems.

the good news is I have a simple fix.

the performance problem is caused by my xul and css using "value="

alecf posted about this problem to n.p.m.performance.  see
news://news.mozilla.org/[e-mail]

I'll go find our bad css and fix it.

moving to .0.8.1, this is the cause of many of our subscribe problems.

Comment 15

[(not reading, please use [e-mail] instead)]

Attachment: the fix

Comment 16

[(not reading, please use [e-mail] instead)]

fix checked in.  sr=bienvenu, a=asa.

Comment 17

[Alec Flett]

sr=alecf

Comment 18

[[e-mail] (gone - use [e-mail] instead)]

Verified fixed.  This is now working on builds: 2001031910 Mandrake 7 with KDE, 
build 2001031910 Mac OS 9.1 and 2001031904 on Windows 2000.