123910 - Get Next 500 news messages shouldn't be displayed when server selected.

Status: RESOLVED FIXED

(SeaMonkey :: MailNews: Message Display, defect)

Description

[Rob Wilcox]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:0.9.8) Gecko/20020204
BuildID:    2002020406

With the newsgroup server selected, the File->Get next 500 news message option
is available, but doesn't do anything.

I figure that this shouldn't be displayed at this point.. It should only be
displayed when an actual newsgroup is selected.

Reproducible: Always
Steps to Reproduce:
1. Open Mail & Newsgroups.
2. Click the newsgroup server at the top of left of the window
3. Click the File menu, you will then see the  menu item displayed.

Actual Results:  The menu item "Get next 500 news messages" is displayed, but
doesn't do anything.

Expected Results:  Maybe it shouldn't be displayed at this time?

Comment 1

[David :Bienvenu]

you mean it should not be enabled? Again, a front end issue...

Comment 2

[Rob Wilcox]

That's right.  I don't think it should be displayed, because clicking it doesn't
appear to do anything at all.
Thx Rob

Comment 3

[grylchan]

adding stephend

Comment 4

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

Rob, Gary; this seems to be a DUP of bug 113414.  Please mark it if you agree,
and I'll verify.

Comment 5

[Rob Wilcox]

No, I don't think it is the same.  What I'm saying is that when you click on the
newsgroup server at the top of the list of newsgroups, then the menu item is
still available.  Is it supposed to be?  When I clicked it, it didn't do anything.
Thx Rob

Comment 6

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

Okay, server vs. newsgroup, my bad.  

Comment 7

[grylchan]

fwiw, 4.x disables get next 500 mesgs when server is selected.

Comment 8

[Rob Wilcox]

Gary how do you mean 4.x ?

Comment 9

[grylchan]

The old netscape communicator browser, currently at version 4.79.

Comment 10

[Rob Wilcox]

Okay, I see.  Then yes, it should not be dispayed as per Communicator  - at
least that's what I think.

Comment 11

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

Should be Seth's, but maybe Sean Su should help?

Comment 12

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

s/should/could, *really* (not a slip of tongue, fingers, rather).

Comment 13

[Stefan Borggraefe]

I have a patch for this.

Comment 14

[Stefan Borggraefe]

Attachment: Patch

Fixes both TB and Seamonkey. This also removes an "else after return" in the
function I touched.

Comment 15

[Stefan Borggraefe]

Comment on attachment 145633 [details] [diff] [review]
Patch

Fairly low-risk patch to fix UI nit.

Comment 16

[Asa Dotzler [:asa]]

Comment on attachment 145633 [details] [diff] [review]
Patch

a=asa (on behalf of drivers) for checkin to 1.7

Comment 17

[Karen Huang]

Verified as fix on latest 1.7 branch 06-24 build.
Changing keywords from fixed1.7 to verified1.7.
Leave this bug status "as is" until this bug be verified on trunk again...