Parameter list of fill() and stroke() lack alpha parameter description

[FrauBoes]

Most appropriate sections of the p5.js website?

Reference

What is your operating system?

Mac OS

Web browser and version

131.0.6778.87 (Official Build) (x86_64)

Actual Behavior

The parameter list is missing the description of the alpha parameter.

The description is present in the source code:

https://github.com/processing/p5.js/blob/1ed240c3b4f9f286010a47d456d216f639f741f5/src/color/setting.js#L1003

Expected Behavior

The description of the alpha parameter should be displayed on the website.

Steps to reproduce

Go to https://p5js.org/reference/p5/fill/ and see the parameter list.

Would you like to work on the issue?

Yes

[ksen0]

Hello @FrauBoes ,

Would you like to work on this? If so you, can prepare a pull request

Description can be from from source code ("optional alpha value, controls transparency (0 - transparent, 255 - opaque).") - or from other pages with an alpha parameter that has a description (Number: alpha value relative to current color range (default is 0-255).)

Besides fill there may be other pages; I see https://p5js.org/reference/p5/stroke/ also has the same issue, though I can't think of others at the moment.

[FrauBoes]

Hi @ksen0,

Happy to fix this! I created a draft PR here: processing/p5.js#7420

Draft PR because I see that the missing description of fill() was fixed recently:

processing/p5.js#7362

However, when checking the live reference page, the alpha parameter still doesn't have a description:

https://p5js.org/reference/p5/fill/

Is the fix not deployed yet?

Either way, my PR adds the description to stroke() and makes another tiny correction (removing the word 'optional' to avoid duplication)

[davepagurek]

Is the fix not deployed yet?

Currently the docs only update when we release a new version of p5. We've been trying to do minor version releases more often so that the docs are more up to date, but it does mean that the docs won't immediately update when a PR into the p5 repo is merged.

[FrauBoes]

Thanks for the info! I started the review of the PR.

[krishna4040]

I think this issue is fixed on the website, and can be closed now