-
-
Notifications
You must be signed in to change notification settings - Fork 82
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
Exercise Solution Discussion #1025
Comments
This works for me! I think the the "suggested solution" should link directly to a p5.js sketch. |
Noting this is a continuation of the discussion in #55 |
So we're going to try the web only tag and implement this! |
@tuantinghuang might as well as an explanation of how to submit an exercise solution on the README as well. |
We're also going to add text based solutions with a "reveal answer" button, the proposed format in Notion is as follows: (This is the solution from #1038) |
One more note is that the exercise p5 sketch embeds that are now marked with "web-only" should be hidden on the chapter pages and let's use another tag as with the text solutions above!
Ok, I think we are all set with this! |
Some update on this:
In the #1053 implementation, instead of using separate |
Summary of current exercise formats:
Text + code with fill in blanks
Text with a linked sketch as part of the prompt
Text only (Open question)
Mockup design for links to solution
If we are to compile an index page for exercise solutions, here's a proposed design of how we can link to the solution from the exercise, similar to how we have "copy/reveal answer." (Question: Does this link to the solution page, or directly to p5 sketch?)
How to format the solution sketches in Notion?
Similarly to how we attach a sketch to an exercise, can we also place the sketch link as a bookmark in the exercise block , and add a label/callout (ex:
web-only
) so it renders as a link ("suggested answer") instead of showing the sketch? Also we can use the label to keep track of what we have only for the website, and not included in the printed version.Something like this maybe?
The text was updated successfully, but these errors were encountered: