Fix missing mouseButton property types by nbogie · Pull Request #8232 · processing/p5.js

@nbogie

Resolves #8231

Changes:

  1. Added a find-replace in patch.mjs to substitute in the correct mouseButton type
mouseButton: { left: boolean; center: boolean; right: boolean };
  1. Added use of the new properties to test/types/basic.ts as regression test.

PR Checklist

  • npm run lint passes
  • [Inline reference] is included / updated. It's already correct for human readers.
  • [Unit tests] are included / updated. Yes. One visual test of fontWeight fails before and after my patch.

@nbogie

@nbogie

@nbogie

@nbogie

After build, I seedocs/parameterData.json has this unrelated addition, which I didn't commit as it's irrelevant to my change:

+    "contrast": {
+      "overloads": [
+        [
+          "Color"
+        ]
+      ]
+    },

@nbogie

davepagurek

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, thanks!

@davepagurek

@allcontributors

@davepagurek

We had trouble processing your request. Please try again later.

@davepagurek

@allcontributors