Skip to content

Fix: Show resizer on "Media & Text" block on unified toolbar mode#10913

Merged
jorgefilipecosta merged 1 commit intomasterfrom
fix/media-and-text-resizer-unified-toolbar
Oct 23, 2018
Merged

Fix: Show resizer on "Media & Text" block on unified toolbar mode#10913
jorgefilipecosta merged 1 commit intomasterfrom
fix/media-and-text-resizer-unified-toolbar

Conversation

@jorgefilipecosta
Copy link
Copy Markdown
Member

@jorgefilipecosta jorgefilipecosta commented Oct 22, 2018

Description

Fixes: #10895
On unified toolbar mode it was not possible to resize the media on Media & Text block.
During the elaboration of #9416 this bug was addressed but I think during subsequent rebases and changes more concretely the usage of our abstracted Resizable box we had a regression.

This PR applies exactly the same fix that was applied in spacer block to the Media & Text block.

How has this been tested?

I checked it is possible to resize the media in Media & Text block in all mode including the unified toolbar.

@jorgefilipecosta jorgefilipecosta added this to the 4.1 - UI freeze milestone Oct 22, 2018
@jorgefilipecosta jorgefilipecosta changed the title Fix: Show resizer onMedia & Text block on unified toolbar mode. Fix: Show resizer on Media & Text block on unified toolbar mode. Oct 22, 2018
@jorgefilipecosta jorgefilipecosta changed the title Fix: Show resizer on Media & Text block on unified toolbar mode. Fix: Show resizer on "Media & Text" block on unified toolbar mode Oct 22, 2018
Copy link
Copy Markdown
Member

@Soean Soean left a comment

Choose a reason for hiding this comment

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

Bug is fixed. Now I am able to resize the image.

@jorgefilipecosta jorgefilipecosta merged commit 74214c0 into master Oct 23, 2018
@jorgefilipecosta jorgefilipecosta deleted the fix/media-and-text-resizer-unified-toolbar branch October 23, 2018 09:17
youknowriad pushed a commit that referenced this pull request Oct 24, 2018
…0913)

## Description
Fixes: #10895
On unified toolbar mode, it was not possible to resize the media on Media & Text block.
During the elaboration of #9416 this bug was addressed but I think during subsequent rebases and changes more concretely the usage of our abstracted Resizable box we had a regression.

This PR applies exactly the same fix that was applied in spacer block to the Media & Text block.


## How has this been tested?
I checked it is possible to resize the media in Media & Text block in all modes including the unified toolbar.
antpb pushed a commit to antpb/gutenberg that referenced this pull request Oct 26, 2018
…rdPress#10913)

## Description
Fixes: WordPress#10895
On unified toolbar mode, it was not possible to resize the media on Media & Text block.
During the elaboration of WordPress#9416 this bug was addressed but I think during subsequent rebases and changes more concretely the usage of our abstracted Resizable box we had a regression.

This PR applies exactly the same fix that was applied in spacer block to the Media & Text block.


## How has this been tested?
I checked it is possible to resize the media in Media & Text block in all modes including the unified toolbar.
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.

Media & Text: Can't resize image with unified Toolbar

2 participants