> On Nov. 20, 2011, 6:53 a.m., Lance Corrimal wrote: > > found another glitch: > > I'm developing scripts in eclipse with lslforge. When I "compile" a script > > in eclipse and then load it using this patch, tabstops are converted into > > IDunnoWhat (it shows as little squares in the script editor). > > > > Ima Mechanique wrote: > I'm not familiar with lslforge, but I'll look into it. Are you sure > they're valid tabs? I ask because it handles tabs from LSL Editor without a > problem. > > Ima Mechanique wrote: > Looks like this is a bug I've introduced recently, as it is also > affecting my LSL Editor created files now. :-( > Trying to track it down, as soon as I can work out how to go back to a > specific revision in Hg. > > Ima Mechanique wrote: > Damn. SL's text editing windows do not like tabs. When copy/pasting the > tabs are converted to spaces, but not when inserting text. I'll look at > finding a way around this. > > Kadah Coba wrote: > I don't remember any in-viewer text input ever liking \t particulary. It > might be best, or at least easier or safer, to just convert them to spaces on > import. > > Cron Stardust wrote: > I use an external editor now for all my scripting (TextWrangler) and I > have set it to use only tabs for all indentation. The viewer accepts it just > fine, and will even regurgitate the tabs when I open the same script in the > external editor sometime later. However, if I try to edit the text using the > in-world editor it will mangle the tabs every time. So my suggestion is > this: don't worry about tabs when saving or loading scripts - leave the text > as-is. If it's too annoying (which I think it is) than a JIRA entry should > be made, unless there is one already, to request that the SL text editor > support real tabs (\t) in the text. That's where the real problem domain is, > not here, IMHO. > > Ima Mechanique wrote: > @Kadah, @Cron. Guys, tabs are now handled as noted it the last diff > (below). Are you suggesting I should remove the conversion? > Personally I like using tabs in my external editors, but hate the boxes > in SL's editor. > > @Cron. What editor is that? I've only tried live editing externally with > LSL Editor, set to use UTF-8 and only indent with tabs, but the tabs only > show as boxes in SL which I find defeats the purpose of clear indentation > (and ugly too).
I had the mistaken impression from my experience that part of that issue was at least the fault of more than just the UI text editor controls. I was only making the point cause the tab issue and the fact that most external editors use \t for tabs, is likely going to lead to "I uploaded a script and now I can't edit it properly in-viewer, thus the uploader is broken". I'd like \t to be preserved on the uploads and simple have a jira regarding the text editor's poor handling of \t linked to STORM-1708, then get that fixed separately. - Kadah ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/516/#review1082 ----------------------------------------------------------- On Nov. 22, 2011, 5:17 p.m., Ima Mechanique wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/516/ > ----------------------------------------------------------- > > (Updated Nov. 22, 2011, 5:17 p.m.) > > > Review request for Viewer. > > > Summary > ------- > > Changes to allow opened script window to save/load to/from files on the users > computer. > > > This addresses bug https://jira.secondlife.com/browse/storm-1708. > > http://jira.secondlife.com/browse/https://jira.secondlife.com/browse/storm-1708 > > > Diffs > ----- > > doc/contributions.txt a1319d553db9 > indra/llui/lltexteditor.h a1319d553db9 > indra/llui/lltexteditor.cpp a1319d553db9 > indra/newview/llfilepicker.h a1319d553db9 > indra/newview/llfilepicker.cpp a1319d553db9 > indra/newview/llfloaternamedesc.h a1319d553db9 > indra/newview/llfloaternamedesc.cpp a1319d553db9 > indra/newview/llpreviewscript.h a1319d553db9 > indra/newview/llpreviewscript.cpp a1319d553db9 > indra/newview/llviewerfloaterreg.cpp a1319d553db9 > indra/newview/skins/default/xui/en/panel_script_ed.xml a1319d553db9 > > Diff: http://codereview.secondlife.com/r/516/diff > > > Testing > ------- > > Successfully opened and saved scripts from/to local files. > > > Thanks, > > Ima > >
_______________________________________________ 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