Skip to content
This repository has been archived by the owner on Dec 11, 2020. It is now read-only.

markdown-to-slides documentation (updates on /worshops/README.md) #17

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

KathyLiu20
Copy link
Contributor

@KathyLiu20 KathyLiu20 commented Mar 31, 2020

Close issue #32

  • updated both the documentation for code review and for developing markdown files into slides in README.md

Issue #55 follow-up

  • updated README.md according to reviewers' comments

@kavuong

Copy link
Contributor

@kavuong kavuong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @KathyLiu20 thank you for your hard work! I've added some comments below.

README.md Outdated

We are specifying these details to help the design team when they put the information into the actual PPT.

First, you need to specify the type -- what slide template you would like to use. Our design team has made many good BitProject templates, and you can find all of them [here](https://docs.google.com/presentation/d/1F5_Qcf_0IduoNKzRUT4owPlE5t5Ab8utmGHUMwZCM_g/edit#slide=id.p). You can make a copy of it and see the details of all templates by adding empty slides.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some headers to designate sections would be helpful here.




Here is a full example of what a slide should be like in a markdown file.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the equivalent of this slide inside of Rochelle's slide template will really drive the point home.

README.md Outdated

# Code Review -Reviewing Markdown Cards

### Checkpoints
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what exactly is the context here? a bit unclear - I think the use of the title "checkpoints" is misleading

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also, markdown cards are being developed based on an existing workshop or tutorial, that should be explained

README.md Outdated
- [ ] Each card has visuals
- [ ] The length of each card is reasonable
- [ ] Logical context progression from card to card
- [ ] Using long sentences
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

long sentences = awkward wording here.

@@ -3,9 +3,11 @@ All of our workshop materials - open for everyone to use!

Our workshops are first written in Markdown, with one .md file corresponding to a slide. Each card must have the type of slide (please consult our Bit brand template) and appropriate labelings of content to be placed on the slide.

# How a Slide should Look
# **Developing Markdown Cards --> PPT That Fits Inside Of Slide Template**
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Context of existing activity cards -> simplified Markdown cards -> PPT slides should be explained.




Since the slides are to help people visualize the ideas better and be like a summary, you should simplify and extract the content from the cards instead of copying it. There should not be long sentences or paragraphs on the slides.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think a specific example of how one can take more in-depth information from an activity and simplify/summarize it for a slide would help the reader understand the expectations




Often, one card contains too much information to be covered in one slide. So you should split them up and make sure the slides are reasonable and understandable.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's worth mentioning that slides have a lot of pictures and we should be mindful of this when creating slides (we don't want to overwhelm the audience with text). With that being said, when converting slides to markdown files, we shouldn't need to include all of the pictures (although still having most would be nice; we definitely want the pictures that further/clarify the concepts)

README.md Outdated

### Documentation for checkpoints

1. The cards should have significant interactive aspect. For example, the BitBloxs games in the Postman Workshop should be included in the activity's cards. In the workshop, we can interact with students directly and verbally, then when we transfer to cards, we should try to create some interaction like steps to guide students through activities.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it would be beneficial to specify how the steps for explaining the interactive aspect differs from regular explanation

Copy link
Contributor

@taimurkashif taimurkashif left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall, the work was really well done! I left a few recommendations for you!

Issue #55
updates on
* documentation on code view
* documentation on cards --> PPT
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants