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

Create a Book block #134

Merged
merged 4 commits into from
Aug 21, 2020
Merged

Create a Book block #134

merged 4 commits into from
Aug 21, 2020

Conversation

vcanales
Copy link
Member

This PR includes changes made to the the dependencies on another PR: #132 and it should land after. I considered both changes to be sufficiently different to require different discussions.

Add a Book block, with a caption that is specific to Book data, and a special placeholder for the cover image.
This is a building block for these future additions:

  • A 3D effect for the book cover, which could allow us to flip the book, or even show a more book like version of the image, like this example
  • A book shelf block that will be used to group book blocks via InnerBlocks. It should display book metadata stored on the book blocks.

Preview

Book1
Book2

Seems like a good idea to try to keep styles as consistent as possible.
`@wordpress/script env` was deprecated; on a fresh clone of this repo,
the deprecation notice doesn't allow starting a new env unless we move
to @wordpress/env.
@jasmussen
Copy link
Member

Love it! Unless the other folks requested want to go with a code review more than I've done, I say let's get this in!

@vcanales
Copy link
Member Author

I'll merge and we can revisit on future revisions! :)

@vcanales vcanales merged commit 04d53e3 into master Aug 21, 2020
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