search

processing / p5.js

p5.js is a client-side JS platform that empowers artists, designers, students, and anyone to learn to code and express themselves creatively on the web. It is based on the core principles of Processing. http://twitter.com/p5xjs —
http://p5js.org/
GNU Lesser General Public License v2.1
21.7k stars 3.34k forks source link

Fix broken links in contributor_docs/documentation_style_guide.md #7220

Open aleannab opened 2 months ago

aleannab commented 2 months ago

Most appropriate sub-area of p5.js?

p5.js version

v1.10.0 (latest)

Web browser and version

Firefox v130.0, though would affect all.

Operating system

Windows, though would affect all.

Steps to reproduce this

Steps:

  1. Go to: https://github.com/processing/p5.js/blob/main/contributor_docs/documentation_style_guide.md
  2. Click on respective links (listed in table below).
  3. Get a 404 error.

Solutions:

Replace listed broken links to the correct links. I did look through the p5.js-website repo, and couldn't find matching file paths.

Source link text: Current link: Suggested link:
src/data/examples/assets https://github.com/processing/p5.js-website/tree/main/src/data/examples ???
src/templates/pages/reference/assets https://github.com/processing/p5.js-website/tree/main/src/templates/pages/reference/assets ???
src/assets/learn https://github.com/processing/p5.js-website/tree/main/src/assets/learn ???
iamniky commented 2 months ago

I'd like to work on this issue

iamnikhilranjan commented 1 month ago

Hi @aleannab, I’m interested in working on this issue. Could you please assign it to me ?

iamnikhilranjan commented 2 weeks ago

Thank you @perminder-17 ! I've opened a pull request within p5.js library repo

Here is the [PR Link] (https://github.com/processing/p5.js/pull/7359)