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

Extra space between paragraph and list when pasting. #1483

Closed
stevebeisner opened this issue Jun 2, 2017 · 2 comments
Closed

Extra space between paragraph and list when pasting. #1483

stevebeisner opened this issue Jun 2, 2017 · 2 comments

Comments

@stevebeisner
Copy link

An extra blank line is inserted when pasting a normal paragraph
followed by a list.

Steps for Reproduction

  1. Starting with an empty text area insert (by keyboard) a normal paragraph
    containing "one", followed by a unordered list containing
    "two".

  2. The innerHTML is now <p>one</p><ul><li>two</li></ul>,
    which displays correctly in the editor.

Start again with an empty text area, only this time, instead of
typing, do a programmatic clipboard.dangerouslyPasteHTML to paste
the same HTML (i.e. <p>one</p><ul><li>two</li></ul>).
Now it displays in the editor with extra lines, and the innerHTML
is now <p>one</p><p><br></p><ul><li>two</li></ul><p><br></p>

Expected behavior:

Whether keyboarding or pasting what was earlier keyboarded, the
result should be the same.

Actual behavior:

See above, under Steps for Reproduction

Platforms:

OSX 10.12, Chrome 58

Version:

Quill 1.2.4

@jhchen
Copy link
Member

jhchen commented Jul 17, 2017

You can now use a new config in 1.3.0 matchVisual to disable Quill's visual paste matching which was causing this.

@jhchen jhchen closed this as completed Jul 17, 2017
@drjackson
Copy link

drjackson commented Jul 25, 2017

It's not working on ordered lists for me. The editor is still adding "<p><br/></p>" before each of them.

  • sorry - this is actually working for me. I thought I was using the latest version of quill, but I wasn't. Didn't realize I was serving it from my packages. Thanks!

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

No branches or pull requests

3 participants