213974 - too much checkbox in subscribe window

Status: VERIFIED FIXED

(SeaMonkey :: MailNews: Message Display, defect)

Description

[Konrad]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; pl-PL; rv:1.5a) Gecko/20030718
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; pl-PL; rv:1.5a) Gecko/20030718

In a window of subscribe i have too much  checkbox to subscribe. I will show
this in attachment in next letter.

Reproducible: Always

Steps to Reproduce:
1. Choose news server and "Subscribe to newsgroup"
2. Type some letter from name of group which interests You (in exmpl.
'pl.comp.ma' to subscribe pl.comp.mail


Actual Results:  
We have only two groups (pl.comp.mail and pl.comp.mail.mta) but much more checkboxes


This bug maybe depend on bug 192437 (please verify)
I see that this bug was absent in Mozilla 1.4 but it is also in Thunderbird

Comment 1

[Konrad]

Attachment: how looks this problem

Comment 2

[Andrew Schultz]

confirmed with linux trunk 20030725
this regressed between linux trunk 2003063005 and 2003070107, probably bug 208093

debug build doesn't show the checkboxes, but still shows a scrollbar when it
isn't needed.  it also asserts.

###!!! ASSERTION: row count did not change by the amount suggested, check
caller: 'rowCount == mRowCount', file nsTreeBodyFrame.cpp, line 1727
###!!! ASSERTION: row count changed unexpectedly: 'mRowCount == rowCount', file
nsTreeBodyFrame.cpp, line 2198

==> XP/Trees

Comment 3

[[e-mail]]

Attachment: Proposed patch

Seth, you didn't fix bug 111321 correctly - you subtracted the two variables
the wrong way around. Otherwise, that would have been the correct fix for bug
92621 in the first place. But this fix uses the new batch system to accurately
reflect what's really going on here.

Comment 4

[[e-mail]]

Back to News.

Comment 5

[Jan Varga [:janv]]

Comment on attachment 128679 [details] [diff] [review]
Proposed patch

r=varga

Comment 6

[David :Bienvenu]

Comment on attachment 128679 [details] [diff] [review]
Proposed patch

sr=bienvenu

Comment 7

[[e-mail]]

Fix checked in.

Comment 8

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified fixed - I just pulled from CVS and built.

news.verizon.net with pl.comp.m doesn't exibit the problem on my Windows 2000 box.