Skip to content

Operators plus#2446

Closed
Kelaphi wants to merge 10 commits intoTurboWarp:masterfrom
Kelaphi:operators-plus
Closed

Operators plus#2446
Kelaphi wants to merge 10 commits intoTurboWarp:masterfrom
Kelaphi:operators-plus

Conversation

@Kelaphi
Copy link
Copy Markdown

@Kelaphi Kelaphi commented Mar 26, 2026

No description provided.

@github-actions github-actions Bot added the pr: new extension Pull requests that add a new extension label Mar 26, 2026
@Kelaphi
Copy link
Copy Markdown
Author

Kelaphi commented Mar 26, 2026

!format

Copy link
Copy Markdown
Contributor

@Brackets-Coder Brackets-Coder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A lot of this extension can either be replicated with TrueFantom's Math extension or other extensions. Some of these blocks can be replicated (with difficulty) in native Scratch without extensions.

I think it's a helpful idea, but you'll also need to add an entry to extensions.json if you want your extension to appear on the gallery. Also, since there's already a similar existing extension, chances of this being merged are lower.

{ blockType: "label", text: Scratch.translate("String") },

{
opcode: "amountofmcdonaldscheeseburgershidinginmybasement",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This opcode is probably unacceptable, try needleInHaystack

getInfo() {
return {
id: "operatorsplus",
name: Scratch.translate("Operators+"),
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
name: Scratch.translate("Operators+"),
name: Scratch.translate("Operators Plus"),

},
},
{
opcode: "replacethebunsinmysandwichwithcatfur",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also unnacceptable opcode, preferably the opcode needs to be a short description of what the block does.

Suggested change
opcode: "replacethebunsinmysandwichwithcatfur",
opcode: "replaceNeedleInHaystack",

{ blockType: "label", text: Scratch.translate("Math") },

{
opcode: "clampmycheeks",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Another odd opcode

},
},
{
opcode: "numberrootthing",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
opcode: "numberrootthing",
opcode: "nthRoot",

return str.slice(Math.max(0, s), Math.max(0, e));
}

clampmycheeks({ N, MIN, MAX }) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

definitely an untolerated opcode.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice banner.

@CubesterYT
Copy link
Copy Markdown
Member

I'm against this extension since other extensions already exist to do most of this

Try contributing to other extensions instead.

@Brackets-Coder
Copy link
Copy Markdown
Contributor

Are we going to close this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: new extension Pull requests that add a new extension

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants