Skip to content
This repository was archived by the owner on Dec 1, 2019. It is now read-only.

Improving floated content layout in nested groups.#994

Merged
pattonwebz merged 4 commits into
WordPress:masterfrom
allancole:fix/nested-alignwidth-blocks-five
Nov 11, 2019
Merged

Improving floated content layout in nested groups.#994
pattonwebz merged 4 commits into
WordPress:masterfrom
allancole:fix/nested-alignwidth-blocks-five

Conversation

@allancole

Copy link
Copy Markdown
Contributor

This PR fixes some of the inconsistencies between the editor and the front end after merging #701.

It allows left/right floated images to look more like what appears in the editor, although it isn’t a perfect 100% match yet. On the frontend, Twenty Twenty resizes the images to make them fit within the layout properly. The editor’s flexible image widths UI, prevent is from replicating the same in the editor. But despite this, the left/right float settings are still respected in both the frontend and backend and so we can iterate on getting an exact 1:1 version later.

Editor Before After
image image image image image image image

@ianbelanger79 ianbelanger79 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This looks good to me, but I definitely want @pattonwebz to test it as well.

@pattonwebz pattonwebz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looking good to me :)

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