Closed FrostyNick closed 10 months ago
Name | Link |
---|---|
Latest commit | 97688fef2d095c08bcc8a96edabe33c64864681d |
Latest deploy log | https://app.netlify.com/sites/projectrhythmswipe/deploys/64d52f17c4cf030008ed9a8a |
Deploy Preview | https://deploy-preview-54--projectrhythmswipe.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
I see the bug you mention. This is caused by #52, in this case the code detects the blue cube 600 pixels before, which means there probably should be a blue cube, or a way to keep track of previous cubes if the number of red/blue cubes change in the future I believe.
Might not be as easy to fix blue cube issue because of the way cube offset was added.
Things should look exactly the same in-game in this update with a few bug fixes, unless any new bugs show up. Many things in code have been changed; pretty much a rewrite of some of the code. I anticipate there will not be a code rewrite at least for the update after this. So probably it's a good time to push this update.
Changes
let xBoardSizeZoomed = 600;
to another value to change the resolution. This value must be set before HTML canvas is created for the resolution to change.Other Important Changes
boardSize
is now split into three variables:boardSize
should always be 600 and doesn't change based on screen resizing; same behavior as before. This represents both the x and y axis.xBoardSizeZoomed
explained in Main Change above. It's the same value as built-inwidth
variable. References to width have been replaced. Docs: https://p5js.org/reference/#/p5/widthyBoardSizeZoomed
- Same value as built-inheight
variable. References to height have been replaced. In theory it could be different thanxBoardSizeZoomed
but it's the same value for now.Known Issues and Fixes
In different resolutions, cops and robbers in start menu are not in the correct x-positions.Fixed.If you have any ideas on improving this PR let me know.