• All submissions to this site are governed by Second Life Project Contribution Agreement. By submitting patches and other information using this site, you acknowledge that you have read, understood, and agreed to those terms.
MAINTENANCE ANNOUNCEMENT - JIRA will undergo maintenance starting 1:00am PDT through 3:00am on Saturday 2010.03.20. Please do not enter issues during this time as the system maybe restarted.
Issue Details (XML | Word | Printable)

Key: VWR-1470
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Low Low
Assignee: Sardonyx Linden
Reporter: Nicholaz Beresford
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
1. Second Life Viewer - VWR

Buttons (IM, Teleport, Profile, ...) in friends list are disabled when opening friends list window

Created: 01/Jul/07 02:00 PM   Updated: 09/Aug/07 03:41 AM
Component/s: User Interface
Affects Version/s: None
Fix Version/s: 1.18.1.2

Time Tracking:
Not Specified

File Attachments: 1. Text File 1470_floater_friends.patch (0.5 kB)


Source Version: 1.17.2
Linden Lab Issue ID: SL-47939
Patch attached: Patch attached


 Description  « Hide

When opening the friends list (clicking "friends" button) the window shows the list of friends, the first entry is selected but the buttons at the bottom of the window (IM, Offer, Teleport) etc. have a disabled state as if no friend was selected in the list. They need to be enabled by clicking an entry, even clicking the same entry (first in list) that is already highlighted when the window opens.



 All   Comments   Work Log   Change History      Sort Order: Ascending order - Click to sort in descending order
Nicholaz Beresford added a comment - 01/Jul/07 02:07 PM

The reason for the behavior is that the function which enables the button (refreshUI) is currently called from LLFloaterFriends::postBuild().

The friend list (scrolllist) has no selected item until the scrolllist object receives the focus (at which time it selects the first).

Postbuild however is called inside buildFloater() before the scrolllist receives the focus (this has been traced through the debugger), so calling refreshUI is pointless.

Solution: I moved refreshUI() from postbuild (which is implicitely called early through buildFloater()) to the constructor after buildFloater() has finished.


Torley Linden added a comment - 10/Jul/07 12:26 PM
Came up in UI triage ( http://wiki.secondlife.com/wiki/Bug_triage/UI_Agenda ), assigned to Sardonyx's care.

Sardonyx Linden added a comment - 13/Jul/07 03:47 PM
Nice fix, thanks!

Dzonatas Sol added a comment - 14/Jul/07 08:31 AM
A patch or contribution from this issue has been applied (and maybe modifed)
to the Open Source Viewer, 1.18.0.6.OS.1. Download it here to test it:
http://sourceforge.net/project/showfiles.php?group_id=191214

Nicholaz Beresford added a comment - 09/Aug/07 03:41 AM

Thanks Bryan!