-
Notifications
You must be signed in to change notification settings - Fork 107
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
How to Download a Web Font and Store It in the Theme - Lesson #887
Comments
Overall feedback and questions:
I think that there is some introductory material missing. Perhaps adding Module 1 of Create a Block Theme (Low-Code) would help make sure we're all starting from the same footing. They can be marked as optional Even though you've created the course as an extension/continuation of Sarah's course, it appears that you've shifted to a purely technical developer audience. Is this assumption correct? The items below are more closely related to the course outline
Specific Feedback
How much of a review is it? How much are you expecting people to know or have retained from previous courses?
Add and export the theme to a file at the end of the title since the module also covers export Be careful when relying on plugins to complete tasks. Every time a plugin is updated the course needs to be reviewed and changes incorporated. If we only rely on core functionality the changes are less frequent and would be covered under reviews for new versions.
If you include manually building a template from template parts, you may also want to include referencing them from
There are two different items that fall under this module:
Which one will you cover?
This use of introduction makes more sense because it is followed by a how to section
Patterns are the only low-code way to provide interntionalization but it's not the only way to do it and using the CLI or coding on Javascript or PHP may be better for the audience you're targeting. |
Hi @caraya, thank you for your feedback. Given that the course outline is detailed on this ticket, which includes the overall module breakdown, and linked issues for separate lessons, I wonder if I could ask you to add your Overall feedback and questions to the main course ticket, and then separate the module/lesson specific feedback on the relevant lesson tickets. That way if we discuss questions pertaining to the course overall vs related to specific lessons, the conversation is tracked in the relevant place. |
OK |
@jonathanbossenger - I'm interested in taking this lesson on. |
Thanks, @justintadlock I appreciate the help. The general scope for this lesson:
Let me know if you have any questions. |
@jonathanbossenger - Just providing a quick update. I've just been following along with your scope, which I think works well. I've wrapped up initial drafts for half of this (bullet points 1 and 2) and expect to have 3 and possibly 4 tomorrow. I may have missed this, but where do we put the content? On the block theme course repo? |
Awesome news, thanks @justintadlock.
I don't mind, you're welcome to add them as PRs to my repo, link to a Google doc here, or whatever is easier for you. |
@jonathanbossenger Thanks. I've forked the repo and added the initial content for Module 5 here: https://github.com/justintadlock/block-theme-course/tree/main/module-05 |
Awesome, thanks @justintadlock |
WP 6.1 Fluid Typography WordPress/gutenberg#39529 |
All 4 of the outlined lessons are up on my fork now: https://github.com/justintadlock/block-theme-course/tree/main/module-05 I plan to spend tomorrow cleaning it up. If anyone has any feedback before (or afterward), that'd be great. |
Thanks, @justintadlock. I'll do my best to give it a quick review later today. |
I've wrapped up my review and copyediting today. I figured it'd be best to give you a chance to review it before submitting a PR. Plus, I need to review how to create and submit a PR. It's been a while... 😆 |
Awesome work, thanks, @justintadlock I'll leave my review notes per lesson as comments here. Lesson 1Spelling/grammar corrections in italics
Also, I wonder if we should say "a recommended location", so as to present this as our suggested "best practice"? |
Lesson 2
It might be worth adding a note as to why you're using the \ character for escaping the quotes here. The requirement to escape the quotes might be new to some learners. |
Lesson 3This is more or a "formalized style guide thing", which we don't have on the training team, and I want to create, but for most of the course lessons I've been using bold to indicate UI elements like clickable items or labels. We probably should formalize this within the training team, so I wouldn't worry about making any changes here, I can always clean this up when the content is merged. |
Thanks for the feedback, @jonathanbossenger. That all looks good, and I'll get it incorporated. I like the use of bold to indicate UI elements. (I'd love to see a standard for that across all of the .ORG style guides. I've mostly used quotes in the past because of the lack of guideline on it.) |
Hmm now that I think about it, I know the docs team has a style guide, perhaps we should all be following that? EDIT: It seems that guide also recommends using Bold when explaining UI to interact with - https://make.wordpress.org/docs/style-guide/formatting/procedures/ FWIW the training team also has a Lesson Plan style guide, which could be applied to course lessons |
Note to self: It might be easier to set the fontFamily to use single quotes inside double-quoted strings. eg:
instead of needing to use the \ escape character. |
Review: Clear step-by-step instructions! |
IMPORTANT
This lesson plan belongs to part of a greater course, Extend a Low Code Block Theme which depends on the completion of Create a Custom Block Theme #1
Please reach out to @jonathanbossenger (Jonathan Bossenger) in the #training team Slack if you would like to help with this lesson plan.
Topic Description
The goal of this lesson is to learn how to download and store custom fonts in a block theme.
Objectives
After completing this lesson, participants will be able to:
Guidelines
Review the [team guidelines] (https://make.wordpress.org/training/handbook/guidelines/)
Review checklist
Reviewers: this is not an extensive list, but it would be helpful if you can keep the following questions in mind while reviewing.
The text was updated successfully, but these errors were encountered: