processing / p5.js-editor

Deprecated desktop editor for p5.js
https://p5js.org
MIT License
338 stars 91 forks source link

Recursion Example in environment should be named "Functions" #157

Closed lmccart closed 8 years ago

lmccart commented 8 years ago

reposting @pippinbar's issue on the p5.js repo:

"The Recursion Example in the downloadable environment (v0.5.2) is an example of using function calls to repeat work, rather than recursion. This example does not seem to appear on the website.

I don't know if this is overly picky, but it might confuse some learners about the nature of recursion (I'm teaching a course that involves some introductory programming)."

the title should be "functions" to match the example on the website http://p5js.org/examples/examples/Structure_Functions.php

lmccart commented 8 years ago

looks like it would be helpful to just do a new pull of the examples from the site. this issue is also related: https://github.com/processing/p5.js/issues/924

antiboredom commented 8 years ago

@lmccart hey - so we're pulling from this file to get those structure examples: https://api.github.com/repos/processing/p5.js-website/contents/examples/examples_src/00_Structure?ref=master

It looks like it references both a recursion and function example. Can you remove the recursion example?

Either way, I've updated the examples and hand-removed that one.

lmccart commented 8 years ago

doh. fixed, put up a real recursion example: http://p5js.org/examples/examples/Structure_Recursion.php