[opensource-dev] Review Request: Fixes paste action in editor windows.
Oz Linden
oz at lindenlab.com
Fri May 11 06:20:26 PDT 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/579/#review1219
-----------------------------------------------------------
indra/llui/lltexteditor.h
<http://codereview.secondlife.com/r/579/#comment1131>
It would be much clearer when reading calls to this method if the parameter were an enum rather than a boolean. Consider the readability difference between:
addLineBreakChar(FALSE);
and
addLineBreakChar(DoNotGroupLines);
- Oz Linden
On May 5, 2012, 8:06 p.m., Ima Mechanique wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://codereview.secondlife.com/r/579/
> -----------------------------------------------------------
>
> (Updated May 5, 2012, 8:06 p.m.)
>
>
> Review request for Viewer.
>
>
> Description
> -------
>
> Fixes paste action in editor windows (notecards, scripts, etc.) so that pasting text is a single action as far as Undo is concerned.
> This is an updating of Satomi Ahn's patch to work with v3.3.1.
>
>
> This addresses bug VWR-20553.
>
>
> Diffs
> -----
>
> doc/contributions.txt 3142ebd0bc5a
> indra/llui/lltexteditor.h 3142ebd0bc5a
> indra/llui/lltexteditor.cpp 3142ebd0bc5a
>
> Diff: http://codereview.secondlife.com/r/579/diff/diff
>
>
> Testing
> -------
>
> Tried pasting/undoing.
>
>
> Thanks,
>
> Ima Mechanique
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20120511/56cfec8d/attachment.htm
More information about the opensource-dev
mailing list