> On March 3, 2011, 11:45 a.m., Richard Nelson wrote: > > indra/llui/lltextbase.cpp, line 604 > > <http://codereview.secondlife.com/r/169/diff/1/?file=1014#file1014line604> > > > > this will still display views that are partially outside of the visible > > rect > > > > what you want to do iscall mDocumentView->draw() inside the block that > > has the clip operation and then hide the mDocumentView when calling > > LLUICtrl::draw. > > > > > > Seth ProductEngine wrote: > The clip rect in the block with drawText() call is not enabled for the > editors without a scroller and even enabling it and calling > mDocumentView->draw() inside that block cuts off the bottom of the last > visible line of text (see screenshot at > https://jira.secondlife.com/secure/attachment/48565/clipped_text.png). > > With allow_scroll="false" & clip_partial="true" settings for the editor > we lose one text line and if there are icons on the first line they will be > clipped anyway because the top of such icons exceeds the top limit of the > visible rect.
With allow_scroll="false" and clip_partial="true", you shouldn't lose a line of text, so that sounds like the bug to fix. If icons on the first line are being clipped, you could increase the v_pad attribute to leave enough room, although the size of the icon should be accounted for in the text layout, so that is either another bug or related to the first bug above. I'd strongly suggest fixing those issues instead of using your proposed fix. Visibility of the text and widgets embedded within the text should be handled in the same way if possible. I'll let you know if I find any obvious source of these problems. - Richard ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/169/#review409 ----------------------------------------------------------- On March 4, 2011, 10:30 a.m., Seth ProductEngine wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/169/ > ----------------------------------------------------------- > > (Updated March 4, 2011, 10:30 a.m.) > > > Review request for Viewer and Richard Nelson. > > > Summary > ------- > > Fixed text editor to display the embedded widgets only if they are in the > currently visible area of a text document. > > > This addresses bug STORM-721. > http://jira.secondlife.com/browse/STORM-721 > > > Diffs > ----- > > indra/llui/lltextbase.cpp 767feb16f05f > > Diff: http://codereview.secondlife.com/r/169/diff > > > Testing > ------- > > > Thanks, > > Seth > >
_______________________________________________ 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
