-
Notifications
You must be signed in to change notification settings - Fork 5
Teacher docs #73
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
Teacher docs #73
Conversation
Guilabel has no style (but this part is in @LucianosLionakis his PR |
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.
Dodona has changed quite a lot apparently since these docs were written. Some stuff will need to be rewritten or removed.
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 would also group the articles in the for-teachers
guide together (like #72).
The dutch page of course management doesn't seem to be rendering here. Can you check what the problem is?
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.
- Move these pages under getting started as with the student docs
- Add a TOC to these pages since the are not a guide and just a collection of topics
- significantly reduce the number of links
Not sure if moving can be done as Lucianos his PR isn't merged yet. |
Issue concerning scrolling, most noticeable when using links between pages. |
Co-authored-by: Rien <[email protected]>
bd57f8e
to
5f7e6e3
Compare
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.
todo in separate pull request:
- split out review to a separate page
- nest the pages to improve sidebar structure
English and Dutch version
Tasks: