Skip to content
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

Beginning of WebGL curves don't have a cap since v1.8 #6816

Open
1 of 17 tasks
davepagurek opened this issue Feb 17, 2024 · 2 comments · May be fixed by #6817
Open
1 of 17 tasks

Beginning of WebGL curves don't have a cap since v1.8 #6816

davepagurek opened this issue Feb 17, 2024 · 2 comments · May be fixed by #6817
Assignees

Comments

@davepagurek
Copy link
Contributor

Most appropriate sub-area of p5.js?

  • Accessibility
  • Color
  • Core/Environment/Rendering
  • Data
  • DOM
  • Events
  • Image
  • IO
  • Math
  • Typography
  • Utilities
  • WebGL
  • Build process
  • Unit testing
  • Internationalization
  • Friendly errors
  • Other (specify if possible)

p5.js version

1.8.0 through to 1.9.0

Web browser and version

Firefox 117

Operating system

MacOS 14.2.1

Steps to reproduce this

Steps:

  1. Draw a curve with beginShape, vertex, and quadraticVertex in WebGL mode
  2. Note that only one end has a cap instead of both
  3. Try setting the version back to 1.7 to see the expected value

Expected:
image

Actual result:
image

Snippet:

function setup() {
  createCanvas(400, 400, WEBGL);
  background(255)
  noFill()
  stroke(0)
  strokeWeight(20)
  translate(-width/2, -height/2)
  beginShape()
  vertex(20, 20)
  quadraticVertex(
    280, 200,
    100, 380
  )
  endShape()
}

https://editor.p5js.org/davepagurek/sketches/ILIzcUpnt

@diyaayay
Copy link
Contributor

May I have a go at this? Would like to work on it.

@davepagurek
Copy link
Contributor Author

Thanks! I'll assign this to you.

@diyaayay diyaayay linked a pull request Feb 17, 2024 that will close this issue
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants