Samuel Gunter

Results 28 comments of Samuel Gunter

You only changed `./library/p5.clickable.js`, but you also need to change: * `./library/p5.clickable.min.js` * `./example/lib/p5.clickable.js` * `./example/lib/p5.clickable.min.js`

Looks like you missed `./example/lib/p5.clickable.min.js`. Also make sure that `./example/lib/p5.clickable.js` is given the exact same code from `./library/p5.clickable.js` since if you just modified the lines you changed in the original...

my bad, i saw [p5.min.js](https://github.com/Lartu/p5.clickable/blob/master/example/lib/p5.min.js) and wasn't thinking correctly, thinking instead it was `p5.clickable.min.js`

Currently, there's a bug (when I run it) where when you click on a button and keep the mouse button down, the cursor resets to the normal cursor.

Looks good to me, except those changes also need to be reflected in the minified and library version also /nit, while we're here plz make the fix mentioned in https://github.com/Lartu/p5.clickable/pull/15...

All of that site seems to be a 404

You can make a pull request for this. Here's some things I would do for this: `this.pointerOnHover = "default";` have the default cursor be "default" since that's what it already...

Click detection is done [as of 2020/08/04] on line 127-128 ```javascript if (mouseX >= this.x && mouseY >= this.y && mouseX < this.x + this.width && mouseY < this.y +...

Can you please provide a [Minimal, Reproducible Example](https://stackoverflow.com/help/minimal-reproducible-example)?

The entire reason I installed `cargo-workspaces` was because cargo doesn't automatically add a new crate to the workspace, and it doesn't even do it. This is a much-needed feature, it's...