----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/474/#review1042 -----------------------------------------------------------
Ship it! thanks, looks good! - Richard On Oct. 4, 2011, 8:11 p.m., Vaalith Jinn wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/474/ > ----------------------------------------------------------- > > (Updated Oct. 4, 2011, 8:11 p.m.) > > > Review request for Viewer and Richard Nelson. > > > Summary > ------- > > Update: would appreciate a LL yes or no to this, as it's one of the issues > that's holding back STORM-64. > > > Texture picker's preview box was never a real widget. It had no XUI presence > so the XUI designers had no control over it. > Also, it's height depended on the it's parent floater's min size which meant > that increasing the min size distorted the preview box, > which must be at a 1:1 ratio. > > I have modified the code in a way that > a) makes the preview box no longer reliant on any of the floater's own > parameters. > b) gives the XUI designers three parameters that affect the preview box > specifically: left, top, and size. > (size is both the size from left to right, and from top to bottom, keeping it > locked to 1:1) > > The changes are only noticeable to coders and xui designers, not the users. > note: It is still not a real widget and it still lacks a follow, layout, > delta and any other widget specific abilities. > > > Diffs > ----- > > indra/newview/lltexturectrl.cpp d36e49ee2651 > indra/newview/skins/default/xui/en/floater_texture_ctrl.xml d36e49ee2651 > > Diff: http://codereview.secondlife.com/r/474/diff > > > Testing > ------- > > The box seems to behave exactly as it did before. > Changing the indicated left, top, and size values in the appropriate xml file > does affect the relevant properties. > > > Thanks, > > Vaalith > >
_______________________________________________ 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