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

formatlesspaste not working correctly in CKEditor #1620

Closed
kdambekalns opened this issue Feb 8, 2018 · 6 comments
Closed

formatlesspaste not working correctly in CKEditor #1620

kdambekalns opened this issue Feb 8, 2018 · 6 comments

Comments

@kdambekalns
Copy link
Member

Description

Formatless paste does not work reliably.

It seem as if CKEditor (and/or our bindings to it) do not allow to completely block the lesaking of markup into Neos through copy and paste. In removeTags.js someone left this comment:

For some reason, it does not strip out <p> Tags; not sure why...

Steps to Reproduce

Using a config like

ui:
  aloha:
    'formatlesspaste':
      'button': FALSE
      'formatlessPasteOption': TRUE

and then pasting something with formatting, the formatting is not removed.

Expected behavior

The formatting is removed as configured.

Actual behavior

The formatting was not removed as expected.

Affected Versions

UI: 1.0.4

@klfman
Copy link
Contributor

klfman commented Mar 14, 2018

Can confirm this bug. Our customers reported unexplicable text overflow. It resulted that they had pasted it from somewhere (Maybe Word?) and there was an inline style with a negative margin-right set.

This inline-style can be cleared with the "remove format" button as a temporary fix.

@dimaip
Copy link
Contributor

dimaip commented Mar 14, 2018

Will try to look into it in a few days, if nobody else does before me.

@klfman
Copy link
Contributor

klfman commented Mar 27, 2018

@dimaip any progress on this topic? It's not exactly urgent but would be nice to have it fixed :)

@dimaip
Copy link
Contributor

dimaip commented Mar 28, 2018

@klfman hey, nope, haven't touched it yet, been fixing other stuff. Feel free to take over if you wish.

@klfman
Copy link
Contributor

klfman commented Apr 5, 2018

@dimaip Sorry, lacking the skill to fix stuff in neos yet :)

@mhsdesign
Copy link
Member

I think we can close this as we now use ckeditor5 and not aloha anymore ^^

but there still seems to be a similar issue #2918

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

No branches or pull requests

4 participants