p5.js-website icon indicating copy to clipboard operation
p5.js-website copied to clipboard

prevent double-slash in asset paths (//assets/ --> /assets/)

Open perminder-17 opened this issue 4 months ago • 3 comments

Resolves #912

I also build the reference again in the website, and commited the regenerated MDX, and included those files in the PR as well.

perminder-17 avatar Aug 02 '25 14:08 perminder-17

Hi @perminder-17 , thanks for this, it looks good. OK to merge!

Not sure if you've come across it, but FYI there's a related, but not quite duplicate (on main not 2.0, and about absolute not relative links), is https://github.com/processing/p5.js-website/issues/878 (see also the comment on that PR)

ksen0 avatar Aug 05 '25 19:08 ksen0

Not sure if you've come across it, but FYI there's a related, but not quite duplicate (on main not 2.0, and about absolute not relative links), is https://github.com/processing/p5.js-website/issues/878 (see also the https://github.com/processing/p5.js-website/pull/892#issuecomment-3133005718)

Just added a comment on that, https://github.com/processing/p5.js-website/pull/892#issuecomment-3176768307 Looks like I found the issue contributor was facing. I can probably help her out :)

perminder-17 avatar Aug 11 '25 20:08 perminder-17

hey @ksen0, just noticed this again in the reference, anything else we need to do here before merging?

davepagurek avatar Dec 03 '25 15:12 davepagurek