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

Draft
codingisfun2831t wants to merge 5 commits from codingisfun2831t-mods into master
codingisfun2831t commented 2026-05-30 15:46:39 -05:00 (Migrated from github.com)

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.

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 commented 2026-05-30 15:55:10 -05:00 (Migrated from github.com)

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

Will try to review soon, but probably won't be available today; thanks for the submission!
e016 commented 2026-05-30 18:58:19 -05:00 (Migrated from github.com)

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.

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 commented 2026-05-31 17:26:55 -05:00 (Migrated from github.com)

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

@codingisfun2831t What in the world is this meant to do? Do you have a screenshot demonstrating this addon at work?
codingisfun2831t commented 2026-05-31 18:49:47 -05:00 (Migrated from github.com)

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)

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 commented 2026-05-31 18:59:10 -05:00 (Migrated from github.com)

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...)

> 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 commented 2026-05-31 22:15:18 -05:00 (Migrated from github.com)

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
I've noticed some other issues. First, it doesn't use the `flatEdge` property that gives flat design blocks its borders: <img width="457" height="249" alt="image" src="https://github.com/user-attachments/assets/6d055a0b-fe83-42a7-b121-8ad3c897f282" /> 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: <img width="485" height="142" alt="image" src="https://github.com/user-attachments/assets/90907e23-553f-4f0e-a312-fb3915f21be6" /> This issue will be amplified in Split, as square dropdowns are always static, and round dropdowns can have blocks on them: <img width="544" height="339" alt="image" src="https://github.com/user-attachments/assets/e52c20fa-d783-4205-aa5e-3d5c49d0f594" />
codingisfun2831t commented 2026-05-31 22:26:14 -05:00 (Migrated from github.com)

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!

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 commented 2026-05-31 22:54:54 -05:00 (Migrated from github.com)

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?

> 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 commented 2026-05-31 23:47:30 -05:00 (Migrated from github.com)

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! 🫪

> > 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 commented 2026-06-02 11:03:40 -05:00 (Migrated from github.com)

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)

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. <img width="915" height="136" alt="image" src="https://github.com/user-attachments/assets/a3299bd1-ac67-403a-902b-4acaa330d468" /> (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 commented 2026-06-02 12:12:06 -05:00 (Migrated from github.com)
image

I got the object blocks to work!!

<img width="190" height="164" alt="image" src="https://github.com/user-attachments/assets/d8cd247a-ce31-48e8-9f26-3fb53d84237f" /> I got the object blocks to work!!
codingisfun2831t commented 2026-06-02 12:24:51 -05:00 (Migrated from github.com)

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

Nice! Would you be able to make a PR for it to my codingisfun2831t-mods branch (or just commit it to that outright)
codingisfun2831t commented 2026-06-07 17:22:05 -05:00 (Migrated from github.com)

@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?

@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 commented 2026-06-07 17:23:15 -05:00 (Migrated from github.com)

@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!

> @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!
PPPDUD commented 2026-08-05 13:30:43 -05:00 (Migrated from github.com)

@codingisfun2831t @e016 Any updates on this?

@codingisfun2831t @e016 Any updates on this?
This pull request is marked as a work in progress.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin codingisfun2831t-mods:codingisfun2831t-mods
git switch codingisfun2831t-mods

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch master
git merge --no-ff codingisfun2831t-mods
git switch codingisfun2831t-mods
git rebase master
git switch master
git merge --ff-only codingisfun2831t-mods
git switch codingisfun2831t-mods
git rebase master
git switch master
git merge --no-ff codingisfun2831t-mods
git switch master
git merge --squash codingisfun2831t-mods
git switch master
git merge --ff-only codingisfun2831t-mods
git switch master
git merge codingisfun2831t-mods
git push origin master
Sign in to join this conversation.
No description provided.