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

wysiwyg is destroying inline styles #5811

Closed
krejcif opened this issue Jul 26, 2016 · 10 comments
Closed

wysiwyg is destroying inline styles #5811

krejcif opened this issue Jul 26, 2016 · 10 comments
Labels
bug report Component: Cms Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development

Comments

@krejcif
Copy link
Contributor

krejcif commented Jul 26, 2016

Preconditions

  1. M2.1 CE & EE

Steps to reproduce

  1. Create CMS block or CMS page
  2. Disable Editor
  3. Insert content
<div style="background:url({{media url="whatever.jpg"}})">
&nbsp;
</div>
  1. Enable editor

Expected result

<div style="background: url('{{media url="whatever.jpg"}}');">&nbsp;</div>

Actual result

  1. <div style="background: url('{{media url=;">&nbsp;</div>
  2. It can destroy look of all page in some conditions

Note

If you use this HTML code in one row, it wont be damaged by editor validation.

@ronnybremer
Copy link

Issue still exists, verified in 2.1.1 CE using both Safari and Firefox ESR 45.3.

Extremely unpleasant during major customization tasks ... Every time you load back a safe block the url get garbled.

@miakusha
Copy link
Contributor

@magexo thank you for your report.
We've created internal ticket MAGETWO-58387 to address this issue.

@miakusha miakusha added the Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development label Sep 16, 2016
@piotrekkaminski
Copy link
Contributor

@miakusha the referenced ticket does not relate any way to this problem

@miakusha
Copy link
Contributor

@piotrekkaminski , oh sorry, correct issue number MAGETWO-58617

mmansoor-magento pushed a commit that referenced this issue Dec 6, 2016
[Firedrakes] Bugfixes:
- MAGETWO-55861 [IE11] User can't change/add new Address From Order page
- MAGETWO-58617 [GitHub] WYSIWYG editor loses data after show-hide editor #5811
@SahakTadevosyan
Copy link

I tried @mmansoorebay solution, but it is not working for me. Any suggestion for this ?

@tomasinchoo
Copy link

I also tried @mmansoorebay solution, but it is not working on 2.1.1.

@gboz17
Copy link

gboz17 commented Jan 24, 2017

I tried @mmansoorebay solution and it does not work on 2.1.3.

@piotrekkaminski
Copy link
Contributor

@gboz17 @tomasinchoo - did you apply this https://github.com/magento/magento2ce/pull/626/files and it did not help?

@pantaoran
Copy link

@piotrekkaminski your link results in a 404 on github

@veloraven
Copy link
Contributor

I'm closing this issue as it was fixed in develop branch and there are no plans to backport it to version 2.1 at the moment.

magento-engcom-team pushed a commit that referenced this issue Jun 17, 2020
[TSG] TESTs Fixes for 2.4.1 (pr51) (2.4-develop)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug report Component: Cms Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development
Projects
None yet
Development

No branches or pull requests