-
-
Notifications
You must be signed in to change notification settings - Fork 3
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
Revamp navigation bar for lessons #18
Comments
At some point it would be nice to have the instructor notes as a top level menu item, if that's where the community wants to put all of its wisdom about teaching a particular lesson. Not needed for this change, but food for thought. |
I have mixed feelings about this. At the moment, the "Extras" item capture everything that isn't a episode neither things that all readers should know about (Licence and Code of Conduct). Instructor Notes is something that is more useful to instructors than learners so I have the feeling that we should have one item "Extras for Learners" and another "Extras for Instructors" but I want use |
Organizationally, I can see why the notes are in extras -- compared to the lesson materials, they are. But I think the notes are (or could be more) important, and it's easy for instructors to not realize that they're there because they're tucked away in the extras tab. So another way to think about the organization of content is that "episodes" is the info for learners + "notes" is info for the instructor + "extras" is everything else. And of course the License + CoC as their own items. |
What about It should indicate to learners that we have one page with references and some extra discussions as well as indicate to instructors that we have some supporting material (instructor notes and page only with figures) that we can expand later to have slides with the figures or slides with the exercises. |
I think this looks great! |
I like this a lot too! This makes the "extra" content more accessible to both instructors and learners |
Suggestion
The text was updated successfully, but these errors were encountered: