-
Notifications
You must be signed in to change notification settings - Fork 99
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
Audit tutorials #432
Audit tutorials #432
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @Ash-2k3, thanks for proofreading, this looks much better! LGTM, just a small note.
One question -- did you want to also "hide" the answers to questions under a dropdown? Or will that be a separate PR? (If the latter, would it be possible to file an issue to track it and try to clean it up soonish? Please link to the issue in your reply to this thread.)
Thanks!
@@ -14,10 +25,10 @@ The UX writing team has filed a request on GitHub asking to update the text on t | |||
|
|||
## Procedure | |||
|
|||
<i>The following steps illustrate how a developer might tackle this issue. Try following this tutorial step-by-step on your local machine! This will give you a better sense of how to tackle other similar issues in the codebase. | |||
*The following steps illustrate how a developer might tackle this issue. Try following this tutorial step-by-step on your local machine! This will give you a better sense of how to tackle other similar issues in the codebase.* | |||
|
|||
**Important:** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you want to use the "IMPORTANT" markdown syntax here, perhaps? https://github.com/orgs/community/discussions/16925
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is fine. (Also we are currently using the Important markdown for practise questions)
Filed an issue for hiding the answers. -#433 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks!
Fixed the format/content of the live tutorials + fixed a broken sidebar link