Skip to content

Conversation

@rakesh2OO5
Copy link

Summary

Fixes incorrect GitHub source links on p5.sound reference pages by routing them to the processing/p5.sound repository instead of processing/p5.js.

Changes

  • Updated reference builder logic to select the correct repository based on module type

Testing

  • Verified generated links point to the correct GitHub repositories

Fixes #1030

@davepagurek
Copy link
Collaborator

Hi! I think this doesn't quite solve the issue.

  • If this is an issue with the 2.x reference, then the PR should be into the 2.0 branch, since this main branch is only for the 1.x site.
  • I think this is just editing the comment in the .mdx source files? You're right that this is also likely incorrect and in need of fixing, but the issue is talking not about a code comment, but about a link in the public site. I believe the source for that is in the ReferenceItemLayout file.

@rakesh2OO5
Copy link
Author

Thanks for the clarification — that helps a lot.

You're right, this issue refers to the 2.x reference site rather than the 1.x site on main, and the visible link likely comes from ReferenceItemLayout rather than the reference builder comment.

I’ll take a look at the 2.0 branch and trace the source of the public link there. I’m happy to close or repurpose this PR if this change isn’t appropriate here.

Thanks again for pointing me in the right direction.

@rakesh2OO5 rakesh2OO5 closed this Jan 2, 2026
@rakesh2OO5
Copy link
Author

Closing this PR — this change targets the reference builder comment on main, but the issue affects the 2.x reference site UI. I’ll follow up with a correct fix on the 2.0 branch.

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.x reference pages for p5.sound functions link to wrong repository

2 participants