Skip to content

Fix Framebuffer Blur example on p5.js 2.x - #1620

Merged
davepagurek merged 1 commit into
processing:mainfrom
bhabishnu:fix/framebuffer-example-2x
Oct 15, 2026
Merged

davepagurek merged 1 commit into
processing:mainfrom
bhabishnu:fix/framebuffer-example-2x

Conversation

@bhabishnu

Copy link
Copy Markdown
Contributor

Resolves #1571

Changes

  • Scope the blur shader with push()/pop() so it doesn't carry into the next framebuffer pass.
  • Scale the spheres for the example's current display size.
  • Retune the focal depth and blur strength for the current camera defaults.

Testing

  • Checked the built site at the current 300×150 embed size through frames 1, 40, 60, and 120.
  • Checked at 800×600 as well.
  • npm run test
  • npm run lint
  • npm run build

AI assistance

AI tools were used as an assistive tool during this change. I reviewed the final diff and verified the behaviour locally.

@davepagurek

Copy link
Copy Markdown
Collaborator

Code changes look good! as a last visual check, do you think you could send a screenshot of what the sketch looks like now when you run your changes locally?

@bhabishnu

Copy link
Copy Markdown
Contributor Author

Yep! Here's what it looks like locally with the current changes:

p5-framebuffer-blur-pr1620

@davepagurek

Copy link
Copy Markdown
Collaborator

awesome, thanks! looks great.

@davepagurek
davepagurek merged commit 225e955 into processing:main Oct 15, 2026
4 checks passed
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.

[p5.js 2.0+ Bug Report]: Framebuffer example breaks on 2+

2 participants