----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/78/#review147 -----------------------------------------------------------
Tested your patch and after testing it works however you do not need to have label_selected at all it works as expected without to - Alexandrea On Jan. 12, 2011, 6:02 a.m., Jonathan Yap wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/78/ > ----------------------------------------------------------- > > (Updated Jan. 12, 2011, 6:02 a.m.) > > > Review request for Viewer. > > > Summary > ------- > > "More" should be "Less" when Media Control is open > > This is a trivial text change in an xml file. The reason I am posting this > here is due to some confusion using label_selected. In this case having it > set to a different value than when label is set to seems to have no effect, > so I have made them identical. > > I scanned all the xml files and there are only about 5 places where > label_selected is different from the preceding label= value. > > Is there any reason to revert back to having them set to different values? > i.e. label="More" and label_selected="Less" > > > This addresses bug storm-844. > http://jira.secondlife.com/browse/storm-844 > > > Diffs > ----- > > doc/contributions.txt 179e0754a27d > indra/newview/skins/default/xui/en/panel_nearby_media.xml 179e0754a27d > > Diff: http://codereview.secondlife.com/r/78/diff > > > Testing > ------- > > > 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
