Skip to content
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

cohort 1, chapters 1-3 #3

Merged
merged 4 commits into from
Feb 1, 2025
Merged

cohort 1, chapters 1-3 #3

merged 4 commits into from
Feb 1, 2025

Conversation

dsollberger
Copy link
Contributor

  • made slides
  • edited "club intro" to link to our textbook
  • added "images" as subdirectory of "slides"

dsollberger and others added 4 commits January 31, 2025 10:13
* made slides
* edited "club intro" to link to our textbook
* added "images" as subdirectory of "slides"
The "1. Preliminaries" title (in the header yaml) is what shows up in the index, but it also automatically acts as the first slide.

I also turned off incremental bulleted lists by default, because I find myself manually turning them off most of the time.

I tried to fix the styling so you wouldn't have to insert the empty spacer columns, because those are a clear sign that something is wrong (you should be able to just put content in your qmd and let styling deal with formatting), but the revealjs style fought me. I might come back to it later.
…Did you preview the slides? I feel like you must have because we discussed them, but what I'm seeing looks broken.
Copy link
Contributor Author

@dsollberger dsollberger left a comment

Choose a reason for hiding this comment

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

On my end, the Quarto callout blocks mostly worked well, but the collapse = "true" function did not work in the slides

Copy link
Member

@jonthegeek jonthegeek left a comment

Choose a reason for hiding this comment

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

I made some tweaks. I removed those .callout-note tags; the formatting was really weird for me, and the one in chapter 3 went off the bottom of the slide and wasn't visible.

@jonthegeek jonthegeek merged commit e0c62b2 into r4ds:main Feb 1, 2025
1 check passed
@dsollberger dsollberger deleted the my-chapter branch February 2, 2025 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants