Skip to content
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

New blocks #4

Closed
wants to merge 3 commits into from
Closed

Conversation

veggiecan0419
Copy link

No description provided.

@veggiecan0419
Copy link
Author

@SharkPool-SP Is this how you wanted it?
Other blocks will still come.

(Formatted as well)

Feel free to edit the code.

@SharkPool-SP
Copy link
Owner

@SharkPool-SP Is this how you wanted it? Other blocks will still come.

(Formatted as well)

Feel free to edit the code.

this still requires the VM edits right

@veggiecan0419
Copy link
Author

@SharkPool-SP Is this how you wanted it? Other blocks will still come.
(Formatted as well)
Feel free to edit the code.

this still requires the VM edits right

@SharkPool-SP Well, it depends what you mean with edits. It doesn't require any changes to TurboWarp/scratch-vm repo, but it does override a VM function (as it should also go off with the vanilla block too, (and Lily/MoreEvents)).

@SharkPool-SP
Copy link
Owner

@SharkPool-SP Is this how you wanted it? Other blocks will still come.
(Formatted as well)
Feel free to edit the code.

this still requires the VM edits right

@SharkPool-SP Well, it depends what you mean with edits. It doesn't require any changes to TurboWarp/scratch-vm repo, but it does override a VM function (as it should also go off with the vanilla block too, (and Lily/MoreEvents)).

just tested, it doesnt work. You cant use startHats on any block with a text/reporter menu

@SharkPool-SP
Copy link
Owner

right so Im not merging this, its done wrong + I dont need the blocks no offense.

I just worked around it myself:
image

@veggiecan0419
Copy link
Author

So how did you do it?

right so Im not merging this, its done wrong + I dont need the blocks no offense.

I just worked around it myself: image

@SharkPool-SP
Copy link
Owner

So how did you do it?

right so Im not merging this, its done wrong + I dont need the blocks no offense.
I just worked around it myself: image

look at the code

@veggiecan0419
Copy link
Author

I think the blocks would still be useful tho

@SharkPool-SP
Copy link
Owner

I think the blocks would still be useful tho

I doubt it, especially since variables and this block can work around it

@veggiecan0419
Copy link
Author

OK.
I'll still see if I can get it to work. If so, and you still don't want it, that's perfectly fine.

@veggiecan0419
Copy link
Author

I think the blocks would still be useful tho

I doubt it, especially since variables and this block can work around it

TurboWarp#639 (comment)
That's why I thought it would be useful

@veggiecan0419
Copy link
Author

I was going to add an any message option to the when broadcast received, and a broadcast name block if you see my original PR.

@SharkPool-SP
Copy link
Owner

Ok

@veggiecan0419 veggiecan0419 deleted the M+ branch May 24, 2024 08:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants