[opensource-dev] Review Request: STORM-236 Actual Code Review
Merov Linden
merov at lindenlab.com
Fri Jan 21 10:04:32 PST 2011
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/113/#review236
-----------------------------------------------------------
indra/newview/llbottomtray.cpp
<http://codereview.secondlife.com/r/113/#comment172>
Unnecessary comment. Please delete.
indra/newview/llbottomtray.cpp
<http://codereview.secondlife.com/r/113/#comment173>
Unnecessary comment. Please delete.
indra/newview/llbottomtray.cpp
<http://codereview.secondlife.com/r/113/#comment174>
What Boroondas said. Please delete that comment.
indra/newview/llbottomtray.cpp
<http://codereview.secondlife.com/r/113/#comment175>
Unnecessary comment.
indra/newview/llbottomtray.cpp
<http://codereview.secondlife.com/r/113/#comment176>
Unnecessary comment.
indra/newview/skins/default/xui/en/menu_bottomtray.xml
<http://codereview.secondlife.com/r/113/#comment177>
Typos: change "selction" to "selection", change "visable" to "visible"
- Merov
On Jan. 20, 2011, 6:37 p.m., Wolfpup Lowenhar wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://codereview.secondlife.com/r/113/
> -----------------------------------------------------------
>
> (Updated Jan. 20, 2011, 6:37 p.m.)
>
>
> Review request for Viewer.
>
>
> Summary
> -------
>
> This allows the Speak Button to auto-hide for those that do not use Voice at all.
>
>
> This addresses bug STORM-236.
> http://jira.secondlife.com/browse/STORM-236
>
>
> Diffs
> -----
>
> doc/contributions.txt 9c7d543fd15d
> indra/newview/llbottomtray.h 9c7d543fd15d
> indra/newview/llbottomtray.cpp 9c7d543fd15d
> indra/newview/llspeakbutton.cpp 9c7d543fd15d
> indra/newview/skins/default/xui/en/menu_bottomtray.xml 9c7d543fd15d
>
> Diff: http://codereview.secondlife.com/r/113/diff
>
>
> Testing
> -------
>
> Built locally and did the following:
> 1 Verified that when Voice is toggled via the preference panel the Speak Button auto hid/showed.
> 2 Verified that drag and drop functionality of the Speak Button was not affected.
> 3 Went to a non-Voice area with Voice active and verified that button was still there but grayed out.
>
>
> Thanks,
>
> Wolfpup
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20110121/b941d050/attachment-0001.htm
More information about the opensource-dev
mailing list