Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Backport of MAGETWO-69152: Removed workaround for old Webkit bug in t… #9655

Merged
merged 1 commit into from
May 19, 2017

Conversation

hostep
Copy link
Contributor

@hostep hostep commented May 16, 2017

…he TinyMCE editor for selections of images and hr elements, because this workaround now causes problems in Google Chrome version 58.

(cherry picked from commit 4d10f05)

Description

This is a backport of issue MAGETWO-69152 for Magento 2.1

Fixed Issues (if relevant)

  1. Chrome version 58 causes problems with selections in the tinymce editor #9518: Chrome version 58 causes problems with selections in the tinymce editor

Reference to the commit on the develop branch:
4d10f05

Discussion

Be aware, this is a slightly different commit then the original one, because on the develop branch we have this commit which was applied: ead58ef, but it isn't applied on the 2.1-develop branch, so there were conflicts in the minified files, please review this again, because I didn't actually test it.

Also: I believe this bug is pretty important to be fixed in Magento 2.0.x as well, are you guys taking care of that?
Also: The same is true for Magento 1.x, can you guys try to communicate this to the M1 support people and try to get it included in the next release of Magento 1.x, and maybe even create a SUPEE-patch for this. Since not every Magento 1 shop will be running on the latest version. For example, we currently maintain around 60 Magento 1 shops which aren't on the very last version (ranging between 1.7.0.2 and 1.9.1.0)

Thanks!

…he TinyMCE editor for selections of images and hr elements, because this workaround now causes problems in Google Chrome version 58.

(cherry picked from commit 4d10f05)
@ishakhsuvarov ishakhsuvarov self-assigned this May 17, 2017
@ishakhsuvarov ishakhsuvarov added this to the May 2017 milestone May 17, 2017
@magento-team magento-team merged commit 27675cd into magento:2.1-develop May 19, 2017
magento-team pushed a commit that referenced this pull request May 19, 2017
magento-team pushed a commit that referenced this pull request May 19, 2017
[EngCom] Public Pull Requests - 2.1
 - MAGETWO-69238: Keep transparency when resizing images [2.1 backport] #9662
 - MAGETWO-69236: Add configurations for change email templates [2.1 backport] #9661
 - MAGETWO-69235: Do not di:compile tests/ folder #9660
 - MAGETWO-69223 Fix for #9646 #9647
 - MAGETWO-69234: Backport of MAGETWO-69152: Removed workaround for old Webkit bug #9655
@magento-team
Copy link
Contributor

@hostep thank you for your contribution

@hostep
Copy link
Contributor Author

hostep commented May 19, 2017

Thanks!

@magento-team or @omiroshnichenko: can you please give some more info about backporting this to Magento 2.0.x and Magento 1.x
So we at least know if this is going to happen or not. And if it is going to happen, could you also provide some estimated timeframe for fixing this in these older Magento versions, that would be very much appreciated! So we can decide if we should put time into this ourselves, or if we can wait for you guys to provide new versions/patches for this issue?

Thanks! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants