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

Add creative commons badge to examples #590

Merged
merged 5 commits into from
Oct 9, 2024
Merged

Add creative commons badge to examples #590

merged 5 commits into from
Oct 9, 2024

Conversation

davepagurek
Copy link
Collaborator

Adds a badge + link to the CC license for examples below the code embed on example pages.

image

@davepagurek davepagurek requested a review from Qianqianye October 8, 2024 20:21
@Qianqianye
Copy link
Collaborator

Thanks @davepagurek!
Hi @xinemata, I will merge this if it looks good to you. I will also update the people page after we have more confirmed plan.

@xinemata
Copy link

xinemata commented Oct 9, 2024

Hi @Qianqianye and @davepagurek, this looks great overall. A minor note I have is if we can break Processing Foundation into its own hyperlink?

For example, <a href="/people/">p5.js Contributors</a> and the <a href="https://processingfoundation.org/people">Processing Foundation</a>

@xinemata
Copy link

xinemata commented Oct 9, 2024

@davepagurek @Qianqianye – actually, since the contributor list on the p5.js repo seems to be more up-to-date than the p5.js people page, maybe we should use this as the default link to attribute p5.js contributors?

@Qianqianye Qianqianye merged commit 1dafaf3 into main Oct 9, 2024
4 checks passed
@Qianqianye Qianqianye deleted the cc branch October 9, 2024 18:54
@Qianqianye
Copy link
Collaborator

Thanks @davepagurek and @xinemata. I updated the link to the p5.js website 'People' page for now after a conversation with @xinemata. Later, we will add the following to the people page:

  • an instruction to add contributors to the list
  • a section for p5.js website design contributors

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.

3 participants