Skip to content

Minimal shape docs #7749

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 26 commits into from
Apr 17, 2025
Merged

Minimal shape docs #7749

merged 26 commits into from
Apr 17, 2025

Conversation

ksen0
Copy link
Member

@ksen0 ksen0 commented Apr 17, 2025

Using abridged list from: #7734 but focusing only on minimal necessary docs to get started:

  • Document bezierOrder()
  • Update bezierVertex() docs
  • Remove quadraticVertex() docs
  • Document splineVertex()
  • Remove curveVertex() (it's already supported by the 1.x compatibility add-on for shapes)
  • Review vertexProperty() docs (sense check)
  • Document splineProperty()
  • Document splineProperties()
  • Update beginShape() docs - reviewed, already OK
  • Update beginContour() docs
  • Document new endShape/Contour(CLOSE) behavior for splines
  • Update endContour(CLOSE) docs
  • Update curve() docs to reflect name change to spline()
  • Remove curveTightness() docs
  • Update curvePoint() docs to reflect name change to splinePoint()
  • Update curveTangent() docs to reflect name change to splineTangent()
  • Update curveDetail() docs, possibly incorporating adapted examples from bezierDetail() docs
  • Remove bezierDetail() docs
  • remove beginGeometry()/endGeometry() docs
  • Update vertex() docs - https://beta.p5js.org/reference/p5/texture/

@ksen0 ksen0 changed the base branch from main to dev-2.0 April 17, 2025 13:06
@ksen0 ksen0 force-pushed the minimal-shape-docs branch from e2cd6d5 to 3554eb3 Compare April 17, 2025 13:34
@ksen0 ksen0 marked this pull request as ready for review April 17, 2025 15:55
@ksen0 ksen0 force-pushed the minimal-shape-docs branch from 66fc232 to 9afdd0e Compare April 17, 2025 16:08
@@ -487,6 +487,9 @@ function loading(p5, fn){
}
};

/**
* @private
Copy link
Member Author

Choose a reason for hiding this comment

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

@davepagurek Actually should this be removed?

@ksen0 ksen0 merged commit 2b13c0d into dev-2.0 Apr 17, 2025
2 checks passed
@ksen0 ksen0 deleted the minimal-shape-docs branch April 17, 2025 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants