Skip to content

add reporter block shapes mod - only color for now#20

Draft
codingisfun2831t wants to merge 5 commits into
masterfrom
codingisfun2831t-mods
Draft

add reporter block shapes mod - only color for now#20
codingisfun2831t wants to merge 5 commits into
masterfrom
codingisfun2831t-mods

Conversation

@codingisfun2831t

Copy link
Copy Markdown
Contributor

I wanted to add object but I couldn't change the layout code.

I'm going to do a clipboard thing next but first I want to fix up wrapFunction with the one I was using in the sparkle class.

@PPPDUD

PPPDUD commented May 30, 2026

Copy link
Copy Markdown
Member

Will try to review soon, but probably won't be available today; thanks for the submission!

@e016

e016 commented May 30, 2026

Copy link
Copy Markdown
Contributor

I think for object-shaped reporters, you can apply a function wrap for fixLayout, moving everything more to the right to make space for the block shape. I'm not sure if this would work well for C-Slots, though.

@PPPDUD

PPPDUD commented May 31, 2026

Copy link
Copy Markdown
Member

@codingisfun2831t What in the world is this meant to do? Do you have a screenshot demonstrating this addon at work?

@codingisfun2831t

Copy link
Copy Markdown
Contributor Author

It just makes the shape for any reporter returning a color (via the new reports type feature in Snap! 12) and then changes that to a rectangle to mimic the color input. It also makes "pen color" have that shaped also (via the same technique as isStop)

@e016

e016 commented May 31, 2026

Copy link
Copy Markdown
Contributor

What in the world is this meant to do?

I think it's supposed to add the block shapes that Snavanced adds (square, "object" shaped, etc...)

@e016

e016 commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

I've noticed some other issues. First, it doesn't use the flatEdge property that gives flat design blocks its borders:

image

Also, it doesn't work in Split! (which isn't surprising)

Square blocks also make it seem like you can fit them into static, read-only slots:

image

This issue will be amplified in Split, as square dropdowns are always static, and round dropdowns can have blocks on them:

image

@e016 e016 marked this pull request as draft June 1, 2026 03:15
@codingisfun2831t

Copy link
Copy Markdown
Contributor Author

Ill fix the flatEdge and Split! support tomorrow, but I dont think the "static, readonly" slots thing really fits. If you think that looks like that, dont use the mod!

@e016

e016 commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

Ill fix the flatEdge and Split! support tomorrow, but I dont think the "static, readonly" slots thing really fits. If you think that looks like that, dont use the mod!

Thanks. I also feel like you could also/instead make string blocks square, as text inputs are also square. Possibly as an option?

@Alessandrito123

Alessandrito123 commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

What in the world is this meant to do?

I think it's supposed to add the block shapes that Snavanced adds (square, "object" shaped, etc...)

Thankfully, I see everything you and others say about my mod on the forums, but these are so interesting that I'm surprised, thanks for giving some feedback to Snavanced! 🫪

@e016

e016 commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

I think the user could choose what type the square block should be used for, so the user can choose between list, color, text, etc...

I personally prefer it when the visuals can easily show what can and can't fit.

image

(Image from EV3 Lab. Data wires have bumps that depend on the data type, that fit or don't fit depending on the shape, it's like a child's shape insertion toy)

@e016

e016 commented Jun 2, 2026

Copy link
Copy Markdown
Contributor
image

I got the object blocks to work!!

@codingisfun2831t

Copy link
Copy Markdown
Contributor Author

Nice! Would you be able to make a PR for it to my codingisfun2831t-mods branch (or just commit it to that outright)

@codingisfun2831t

Copy link
Copy Markdown
Contributor Author

@d016 Your version of the mod looks good, but why is it a seperate file? Would I be able to change it to reporter-block-shapes.js?

@e016

e016 commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

@d016 Your version of the mod looks good, but why is it a seperate file? Would I be able to change it to reporter-block-shapes.js?

whoops... you can change it!

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.

4 participants