----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/474/#review1032 -----------------------------------------------------------
indra/newview/lltexturectrl.cpp <http://codereview.secondlife.com/r/474/#comment1046> What kind of safety does this provide? indra/newview/skins/default/xui/en/floater_texture_ctrl.xml <http://codereview.secondlife.com/r/474/#comment1045> For more clarity, replace the work 'it' by the name of whatever you're describing here. (Probably 'The texture picker'.) - Boroondas On Sept. 16, 2011, 2:17 a.m., Vaalith Jinn wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/474/ > ----------------------------------------------------------- > > (Updated Sept. 16, 2011, 2:17 a.m.) > > > Review request for Viewer. > > > Summary > ------- > > 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