SharkPool/Camera.js -- Add touch precision block, optimize, and improve code#2411
SharkPool/Camera.js -- Add touch precision block, optimize, and improve code#2411GarboMuffin merged 8 commits intomasterfrom
Conversation
|
!format |
|
Okay, I'll look into it when I have time |
Brackets-Coder
left a comment
There was a problem hiding this comment.
Quick skim. Code looks better now, I just have to find time to test and make sure everything's still working
I don't have any older projects with this extension so I can't do much regression testing :/
|
!format |
|
@Brackets-Coder Use this sb3 (rename .zip to .sb3) which has the old version. Try it out, notice the jitter. New version of the extension is more optimized, and enabling precise touch mode fixes the jitter. |
|
i cant seem to unbind this sprite from the default camera, even though the block suggest i could |
I could change that in this update |
alr |
I just hid the block since this block is technically just a bind block to the default camera. |
|
!format |
|
The formatting bot didn't find any formatting issues. It currently only checks the extensions folder. The author or a maintainer can run terminal command 'npm run format' manually to format all files. |
alr, ig |
RedMan13
left a comment
There was a problem hiding this comment.
i cant see any issues, and the changes seem perfectly fine to me
I'll get to you as soon as I can, may be a few days |
Few days later |
|
I'll take a look now, thanks for the reminder |
There was a problem hiding this comment.
Aside from the sketchy platformer physics provided in the demo, the most recent commit of the camera extension looks good. Enabling the new touch precision block entirely mitigates the sketchy physics and makes it buttery smooth. Nice job.
- I tested all modified functionality and found no bugs or unintuitive behavior.
- I tested the provided demo project with the newly added block, I haven't yet manually confirmed that the other blocks (zoom, direction, etc). are still working as expected.
- I believe a well-formed project using an older version of this extension will not be broken by this new version.
- I have no additional concerns.
Below you can see that the player "bounces" off of the wall when constantly pressing D to move right into it. This only happens with the new precision mode enabled. It's better than clipping through the wall or bouncing to the top (which happens without it) but I was wondering if you could make the player not move in such circumstances? Is this a project limitation or extension limitation? I'm not familiar with how this extension works, so to be honest I'm not the best one to be reviewing this
video.webm
That is not entirely an issue with the extension, it still happens when the zoom is at the default 100. You can barely see it but it does happen. It becomes more visible because of non-rounded camera xy values multiplied by the camera zoom |
|
!format |
|
@SharkPool-SP can it wait until next week? |
Idk |
Vro 🙏 |
DogeisCut
left a comment
There was a problem hiding this comment.
- I tested all modified functionality and found no bugs or unintuitive behavior.
- I believe a well-formed project using an older version of this extension will not be broken by this new version.
- I have no additional concerns.
Thx for the reminder I'll get it sometime this afternoon or tomorrow afternoon |
|
yolo |
|
Nevermind I won't get it this afternoon then |
Garbs finally understands life 🥹 |

Uh oh!
There was an error while loading. Please reload this page.