Skip to content
This repository has been archived by the owner on Jun 26, 2020. It is now read-only.

Fixed image resize flashing unexpected size #334

Merged
merged 13 commits into from Jan 3, 2020
Merged

Fixed image resize flashing unexpected size #334

merged 13 commits into from Jan 3, 2020

Conversation

mlewand
Copy link
Contributor

@mlewand mlewand commented Nov 15, 2019

Suggested merge commit message (convention)

Fix: Fixed image resize behavior upon short clicking a handle without dragging. Image will no longer became full width, nor will it briefly flash an unexpected size. Closes ckeditor/ckeditor5#5189 and closes ckeditor/ckeditor5#5195.

MINOR BREAKING CHANGE: Resizer options object now also takes an editor instance.

MINOR BREAKING CHANGE: Removed the downcastWriter property from the ResizerOptions interface.


Additional information

Subpr of ckeditor/ckeditor5-widget#112.

@mlewand mlewand added the pr:sub label Nov 15, 2019
@coveralls
Copy link

coveralls commented Nov 15, 2019

Coverage Status

Coverage decreased (-1.8%) to 97.883% when pulling df798bb on i/5189 into 6fe524c on master.

@Reinmar
Copy link
Member

Reinmar commented Nov 20, 2019

R- as you can indeed remove options.downcastWriter.

@mlewand
Copy link
Contributor Author

mlewand commented Jan 2, 2020

Added test coverage for changes in widget plugin (widget toolbar visibility).

@Reinmar Reinmar merged commit 9148013 into master Jan 3, 2020
@Reinmar Reinmar deleted the i/5189 branch January 3, 2020 10:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
3 participants