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

Improving floated content layout in nested groups. #994

Merged

Conversation

allancole
Copy link
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

Copy link
Contributor

@ianbelanger79 ianbelanger79 left a comment

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.

Copy link
Member

@pattonwebz pattonwebz left a comment

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