search

mwieder / glx2ScriptEditor

glx2ScriptEditor is an alternative script editor for the LiveCode IDE
MIT License
6 stars 3 forks source link

on mouseup - complete on Return insert some strange text #3

Closed matthiasrebbe closed 3 years ago

matthiasrebbe commented 4 years ago

When pressing return after on mouseonp not only end mouseup is added, but also this text -- replace this with your own code throw param(0) && "has not yet been implemented"

can this be changed somewhere or is coded in the script?

mwieder commented 4 years ago

There's a preference for Test-driven Development on the preferences panel. I thought I had the default set to false, but apparently that's not so. Check the checkbox, then uncheck it again and it should be turned off. I'll put changing the default on my todo list.

mwieder commented 4 years ago

Closed by accident.

mwieder commented 4 years ago

Again. Sheesh.

matthiasrebbe commented 4 years ago

That is not working. Did as you suggested. Checked, unchecked that "test-driven.." option. Still getting that text after pressing Return when "complete on Return" is enabled.

mwieder commented 4 years ago

Well, that's disturbing - it does work here, but I don't suppose that helps you much. Is there an entry near the bottom of your "GLX2 Code Prefs.txt" file for "uEncourageTDD", and is the value false?

mwieder commented 4 years ago

I just looked through the code, and the default is indeed "false", so you shouldn't have that happen unless the value explicitly gets set.

matthiasrebbe commented 4 years ago

This is the entry <uEncourageTDD>false</uEncourageTDD>, so it seems to be correct. But it still shows the behavior.

I am on Mac OS

matthiasrebbe commented 4 years ago

Mark, by any chance, is it possible that for mouseup there is something left here group "GLX2 Default Handler" of Cd "defaults main cd" of stack "revGLX2Defaults"

and here

button "my default handler" of cd "glx2Behaviors" of stack "revGLX2 default handlers master"

At least in both scripts there is a preconfigured mouseup handler like this

on mouseUp pMouseBtnNum
   -- replace this with your own code
   throw param(0) && "has not yet been 
end mouseUp

while the other mouse.. handlers in those scripts are empty.

mwieder commented 4 years ago

Thanks, yes. Fixed now. That was leftover test code, and very specific to just new mouseUp handlers.

matthiasrebbe commented 4 years ago

The new version 4.12 build 2020.05.24 still adds the testcode to new mouseup handlers.

mwieder commented 4 years ago
ok - I'll go look at that again. I probably overwrote something.
mwieder commented 4 years ago

Yep - I did indeed overwrite something. Will be fixed soon.

mwieder commented 3 years ago

As far as I can tell, this is fixed now with version 4.1.7.