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

Add a 50ms timeout after document#focus event before rendering #1201

Merged
merged 1 commit into from
Dec 12, 2017

Conversation

ma2ciek
Copy link
Contributor

@ma2ciek ma2ciek commented Dec 11, 2017

Suggested merge commit message (convention)

Fix: Added a 50ms timeout after Document#focus event before rendering. Closes ckeditor/ckeditor5#676. Closes ckeditor/ckeditor5#4192. Closes ckeditor/ckeditor5#4190. Closes ckeditor/ckeditor5#4188.

@ma2ciek ma2ciek requested a review from szymonkups December 11, 2017 15:40
@szymonkups szymonkups requested a review from Mgsy December 12, 2017 07:53
@Mgsy
Copy link
Member

Mgsy commented Dec 12, 2017

It also fixes this issue - https://github.com/ckeditor/ckeditor5-engine/issues/1155.

Copy link
Member

@Mgsy Mgsy left a comment

Choose a reason for hiding this comment

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

Works good 👌

@ma2ciek
Copy link
Contributor Author

ma2ciek commented Dec 12, 2017

It should also fix https://github.com/ckeditor/ckeditor5-engine/issues/1153. Could you check it too @Mgsy? I've tested it on the article sample.

@Mgsy
Copy link
Member

Mgsy commented Dec 12, 2017

It should also fix #1153. Could you check it too @Mgsy? I've tested it on the article test.

I've checked it and this PR fixes mentioned issue.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants