[opensource-dev] Review Request: STORM-1459 "Wearing Tab" - Add ability to copy displayed inventory names to clipboard

Lance Corrimal Lance.Corrimal at eregion.de
Thu Jul 7 07:55:01 PDT 2011



> On June 30, 2011, 11:02 a.m., Vadim ProductEngine wrote:
> > indra/newview/llpanelwearing.cpp, lines 297-302
> > <http://codereview.secondlife.com/r/370/diff/3/?file=3241#file3241line297>
> >
> >     Looks a bit confusing.

but it's elegant!


- Lance


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/370/#review834
-----------------------------------------------------------


On June 30, 2011, 1:27 a.m., Jonathan Yap wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://codereview.secondlife.com/r/370/
> -----------------------------------------------------------
> 
> (Updated June 30, 2011, 1:27 a.m.)
> 
> 
> Review request for Viewer.
> 
> 
> Summary
> -------
> 
> Add a feature on the "Wearing TAB" where users could copy to the clipboard everything you see in the "Wearing TAB". This would make the blogging communities life soooooo much easier instead of having to type out all that information.
> 
> The label on this button needs input from someone on the XD team.
> 
> I would like to know if my code for adding a CR to the end of every line but the last one could be done in a more elegant way.
> 
> 
> This addresses bug STORM-1459.
>     http://jira.secondlife.com/browse/STORM-1459
> 
> 
> Diffs
> -----
> 
>   doc/contributions.txt f9864a43ddf0 
>   indra/newview/llpanelwearing.h f9864a43ddf0 
>   indra/newview/llpanelwearing.cpp f9864a43ddf0 
>   indra/newview/skins/default/xui/en/menu_wearing_gear.xml f9864a43ddf0 
> 
> Diff: http://codereview.secondlife.com/r/370/diff
> 
> 
> Testing
> -------
> 
> Clicked on Send to Clipboard button and was able to paste results into an editor.
> 
> 
> Thanks,
> 
> Jonathan
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20110707/37004d25/attachment.htm 


More information about the opensource-dev mailing list