too much checkbox in subscribe window

VERIFIED FIXED in mozilla1.5beta

Status

SeaMonkey
MailNews: Message Display
--
major
VERIFIED FIXED
14 years ago
9 years ago

People

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

Tracking

({regression})

Trunk
mozilla1.5beta
x86
Linux
regression

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

14 years ago
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
(Reporter)

Comment 1

14 years ago
Created attachment 128574 [details]
how looks this problem

Comment 2

14 years ago
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
Assignee: sspitzer → varga
Status: UNCONFIRMED → NEW
Component: Subscribe → XP Toolkit/Widgets: Trees
Ever confirmed: true
Keywords: regression
Product: MailNews → Browser
QA Contact: stephend → shrir

Comment 3

14 years ago
Created attachment 128679 [details] [diff] [review]
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.

Updated

14 years ago
Attachment #128679 - Flags: superreview?(sspitzer)
Attachment #128679 - Flags: review?(varga)

Comment 4

14 years ago
Back to News.
Assignee: varga → sspitzer
Component: XP Toolkit/Widgets: Trees → Subscribe
Product: Browser → MailNews
QA Contact: shrir → stephend
Target Milestone: --- → mozilla1.5beta

Comment 5

14 years ago
Comment on attachment 128679 [details] [diff] [review]
Proposed patch

r=varga
Attachment #128679 - Flags: review?(varga) → review+

Comment 6

14 years ago
Comment on attachment 128679 [details] [diff] [review]
Proposed patch

sr=bienvenu
Attachment #128679 - Flags: superreview?(sspitzer) → superreview+

Comment 7

14 years ago
Fix checked in.
Status: NEW → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
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.
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey

Updated

9 years ago
Component: MailNews: Subscribe → MailNews: Message Display
QA Contact: stephend → search
You need to log in before you can comment on or make changes to this bug.