307923 - Highlighted users on channel user list are lost when switching channels

Status: RESOLVED FIXED

(Other Applications :: ChatZilla, enhancement)

Description

[Mark Micallef]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.7.10) Gecko/20050716 Firefox/1.0.6
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.7.10) Gecko/20050716 Firefox/1.0.6

Problem one

Open 2 channels.
Select a user from first channel.
Switch to another channel.
Switch to first channel.
The user that was selected is not highlited any more.

Problem 2

Open a channel with enough number of users - larger than the number that can
fill the channel window - where users can be scrolled either using the up and
down arrows or the scroll bar.

Scroll down slowly using one method method at a time.
If the channel is busy - messages in main windows and user joing and leaving the
channel - with every refresh the scrolling bar returns to the uppermost position
and the user being highlited is resetted to the first one on the list.



Reproducible: Always

Comment 1

[:Gijs (he/him)]

Problem 2 is a duplicate of bug 271989.
Please, the next time you file bugs, one problem per bug :-). Otherwise 
developers don't know where to find things. Also search for duplicates before 
you file a new bug.
Confirmed bug on latest XULRunner nightly, and with James Ross' developer 
version of CZ. I don't think any related code is different in 0.9.68.5, so I'm 
pretty sure this is a valid bug :-).
Adjusting summary so this isn't a dupe anymore.

Comment 2

[:Gijs (he/him)]

Attachment: [checked in] Patch that saves and restores the selection.

This patch saves and restores the selection on the channel object. The scroll position thing is a thug, I need more time for that. So for now, let's just fix this bit first.

Comment 3

[James Ross]

Severity --> ENH, since the behaviour of clearing the selection is deliberate.

Comment 4

[:Gijs (he/him)]

I'm resolving this bug as FIXED, as it fixes the problem in the summary of the bug. The scrolling problem is also described in bug 271989, and it seemed useless to have both around.