----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/276/ -----------------------------------------------------------
Review request for Viewer and Richard Nelson. Summary ------- Reason: When an undocked side tray tab floater got closed with Ctrl+Shift+W, LLSideTray::setTabDocked() was called. It docked the floater but didn't update the dock/undock icon. Fix: Made setTabDocked() a general purpose method, not a hack suitable for using only as a floater close callback in the basic viewer mode. It now updates the dock/undock icon. Other changes: * Replaced numerous calls to toggleTabDocked with setDocked(), that is safer because does exactly what you want. * Got rid of a duplicated floater close callback. This addresses bug STORM-1093. http://jira.secondlife.com/browse/STORM-1093 Diffs ----- indra/newview/llfloatersidetraytab.cpp UNKNOWN indra/newview/llsidetray.h UNKNOWN indra/newview/llsidetray.cpp UNKNOWN Diff: http://codereview.secondlife.com/r/276/diff Testing ------- Testing (un)docking side tray tabs in Basic and Advanced viewer modes. Thanks, Vadim
_______________________________________________ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges