----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/370/#review834 -----------------------------------------------------------
Ship it! No major objections. indra/newview/llpanelwearing.cpp <http://codereview.secondlife.com/r/370/#comment852> Looks a bit confusing. - Vadim 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 > >
_______________________________________________ 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
