-
Notifications
You must be signed in to change notification settings - Fork 109
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
Uncovering the Cover Block #913
Comments
This comment was marked as outdated.
This comment was marked as outdated.
@artdecotech has volunteered to review this. |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
Thank you very much @amitpatel0702 |
Review: CTA example |
WordPress/gutenberg#41153 6.3 |
I plan to update and use this tutorial in the learning pathway project. |
@westnz I am giving myself a due date of June 7th for this lesson. |
Lesson Development Checklist
|
This comment was marked as outdated.
This comment was marked as outdated.
Another review for contributor day today, and here is my comment. |
Tutorial/Lessons Review Checklist
Great tutorial, @westnz. The hands-on demos are very cool. There's one a11y issue with the hero video example: it defaults to The problem is caused by how the block was coded in Gutenberg (see this open issue), but I think it's worth pointing out in the lesson. |
Thank you for the feedback folks @ironnysh, I think I am going to use another example instead of the video banner then. |
LP.Uncovering.the.Cover.block.mp4 |
Added a new version and re-recorded the first example. |
//published |
//lesson
The text was updated successfully, but these errors were encountered: