----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/244/#review589 -----------------------------------------------------------
Ship it! Looks good. Some minor stuff still: indra/newview/llviewerchat.h <http://codereview.secondlife.com/r/244/#comment529> Why repeat the 'public:' ? indra/newview/llviewerchat.h <http://codereview.secondlife.com/r/244/#comment528> Indented with 4 spaces instead of 1 tab like the other lines. doc/contributions.txt <http://codereview.secondlife.com/r/244/#comment526> doc/contributions.txt <http://codereview.secondlife.com/r/244/#comment527> I guess the STORM-1077 and STORM-1019 entries were added in error? Or why are you replacing them, rather than just adding STORM-1095? - Boroondas On April 13, 2011, 6:40 a.m., Jonathan Yap wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/244/ > ----------------------------------------------------------- > > (Updated April 13, 2011, 6:40 a.m.) > > > Review request for Viewer. > > > Summary > ------- > > Chat preferences > font size should increase size of input text in the chat > box > > > This addresses bug STORM-1095. > http://jira.secondlife.com/browse/STORM-1095 > > > Diffs > ----- > > doc/contributions.txt a8f868007986 > indra/llui/lllineeditor.h a8f868007986 > indra/llui/lllineeditor.cpp a8f868007986 > indra/newview/llbottomtray.cpp a8f868007986 > indra/newview/llfloaterpreference.cpp a8f868007986 > indra/newview/llnearbychatbar.h a8f868007986 > indra/newview/llnearbychatbar.cpp a8f868007986 > indra/newview/llviewerchat.h a8f868007986 > indra/newview/llviewerchat.cpp a8f868007986 > > Diff: http://codereview.secondlife.com/r/244/diff > > > Testing > ------- > > Change font size in preferences and see > 1) Font size in chat input box changes to new size immediately > 2) Font size is set to selected size when viewer is restarted > > > 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