-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Label atomic procedures as "define (no refresh)" #26
Label atomic procedures as "define (no refresh)" #26
Conversation
src/blocks/Block.as
Outdated
@@ -41,6 +41,7 @@ import flash.filters.GlowFilter; | |||
import flash.geom.*; | |||
import flash.net.URLLoader; | |||
import flash.text.*; | |||
import flash.external.ExternalInterface; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's this used for?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That was for console.logging and I forgot to remove it. Whoops
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wow, not using DialogBox.notify
as your main logging system!? 🌈
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
@@ -594,6 +596,7 @@ public class Block extends Sprite { | |||
dup.parameterNames = parameterNames; | |||
dup.defaultArgValues = defaultArgValues; | |||
dup.warpProcFlag = warpProcFlag; | |||
dup.setSpec(newSpec); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there any reason for this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep! Otherwise "no refresh" doesn't show up in the scrap block pane thingy script browser because the blockspec never gets reloaded when its in that pane, just duplicated.
Resolves #21.