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 describe() to default sketch #1954

Open
catarak opened this issue Nov 3, 2021 · 4 comments
Open

Add describe() to default sketch #1954

catarak opened this issue Nov 3, 2021 · 4 comments
Labels
Area:Accessibility Category for accessibility related features and bugs Feature Request

Comments

@catarak
Copy link
Member

catarak commented Nov 3, 2021

Nature of issue?

  • New feature request

New feature details:

As suggested in processing/p5.js#5427, adding describe() would hopefully encourage folks to add a description to their sketches. We could also think about having different descriptions for each of the different translations.

@nickmcintyre
Copy link
Member

@catarak I got started on this issue but am not too sure how translations work -- having descriptions in different languages is a great idea!

I think I should read up on i18next and figure out how to apply it along the lines of the following snippet.

import i18next from 'i18next';

const description = i18next.t('DefaultSketch.Description');

const defaultSketch = `function setup() {
  createCanvas(400, 400);
}

function draw() {
  background(220);
  describe('${description}');
}`;

Does that seem like a route forward? Happy to test with /translations/locales/es-419/translations.json.

@kjhollen kjhollen added Area:Accessibility Category for accessibility related features and bugs Feature Request labels Nov 12, 2021
@parteekcoder
Copy link
Contributor

hey @raclim is this issue still open to work as there is one PR

@parteekcoder
Copy link
Contributor

hey @raclim can you please describe about this issue more as It is not clear from above discussion what is exactly required

@Qianqianye
Copy link
Collaborator

Thanks @parteekcoder for your interest in working on it.
Let's revisit this issue after the describe() guideline is completed in processing/p5.js#6387

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area:Accessibility Category for accessibility related features and bugs Feature Request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants