[opensource-dev] Review Request: STORM-1911: Go-to line function for the internal LSL script editor
Kelly Washington
kelly at lindenlab.com
Wed Sep 19 15:31:23 PDT 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/596/#review1266
-----------------------------------------------------------
Ship it!
Ship It!
- Kelly Washington
On Sept. 19, 2012, 7:26 a.m., MartinRJ Fayray wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://codereview.secondlife.com/r/596/
> -----------------------------------------------------------
>
> (Updated Sept. 19, 2012, 7:26 a.m.)
>
>
> Review request for Viewer.
>
>
> Description
> -------
>
> Repository is here: https://bitbucket.org/MartinRJ/storm-1911
>
> I more or less cloned the search-function and modified the floater via xml (new file: floater_goto_line.xml) to fit for a 'go to line' floater.
> Also I added a callback to prevalidate the input in the 'go to line' line-editor, so that only numbers can be entered into it.
>
>
> This addresses bug STORM-1911.
> https://jira.secondlife.com/browse/STORM-1911
>
>
> Diffs
> -----
>
> doc/contributions.txt 7a056d7afeb2
> indra/newview/CMakeLists.txt 7a056d7afeb2
> indra/newview/llfloatergotoline.h PRE-CREATION
> indra/newview/llfloatergotoline.cpp PRE-CREATION
> indra/newview/llpreviewscript.h 7a056d7afeb2
> indra/newview/llpreviewscript.cpp 7a056d7afeb2
> indra/newview/skins/default/xui/en/floater_goto_line.xml PRE-CREATION
> indra/newview/skins/default/xui/en/panel_script_ed.xml 7a056d7afeb2
>
> Diff: http://codereview.secondlife.com/r/596/diff/
>
>
> Testing
> -------
>
> Tested on my local PC with a modded "3.3.4-release3" version.
> Open a script with at least 300 lines inside inventory, go to menu 'Edit-> go to line', enter a number less than 300 into the 'Go to' - field. Expected result: the cursor jumps to the entered line number (column 0).
> Repeat the test with a script inside an object's content.
>
>
> Thanks,
>
> MartinRJ Fayray
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20120919/78a7557c/attachment.htm
More information about the opensource-dev
mailing list