Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added P_2_2_5_02 #54

Merged
merged 1 commit into from
Jul 6, 2016
Merged

Added P_2_2_5_02 #54

merged 1 commit into from
Jul 6, 2016

Conversation

Niels-NTG
Copy link
Member

Port of P_2_2_5_02.pde

I have some ideas on how to improve this particular sketch. Currently only 2 of the 11 svg files are used in the sketch. It might be fun to have a key that randomly picks a set of two SVG's from an array of SVG files.

@b-g
Copy link
Member

b-g commented Feb 16, 2016

Sure, but please make sure that it reproduces the old behaviour ... But the bigger picture with SVG is for me, that currently in p5js a SVG just becomes an pixel image and it becomes no longer ad-hoc modifiable like as it is in processing e.g. you can't change the fill color dynamically. A lot of the SVG as modul idea relies an this feature. Not sure how to deal with it.

@Niels-NTG
Copy link
Member Author

The Processing Foundation and collaborators are actively working on full SVG support. Zeno Zeng from the Processing Foundation currently has a library for p5 that adds SVG support. I think they intend to eventually merge with p5.

@Niels-NTG
Copy link
Member Author

Until then I will just keep the sketch as it currently is. We can re-evaluate it at a later point.

@b-g
Copy link
Member

b-g commented Feb 17, 2016

👍! Good to know!

@b-g b-g merged commit a837e5b into generative-design:master Jul 6, 2016
@b-g
Copy link
Member

b-g commented Jul 6, 2016

@Niels-NTG Thanks! 👍

@Niels-NTG Niels-NTG deleted the pr/25 branch July 6, 2016 14:07
b-g added a commit that referenced this pull request Jan 8, 2018
added encapsulate.js to libs & npm scripts
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.

2 participants