search

Rosey / markdown-draft-js

A tool to convert content created in DraftJS to markdown and vice versa.
https://rosey.github.io/markdown-draft-js/
MIT License
318 stars 69 forks source link

The order list with index error #127

Closed FerrisLee-180505 closed 3 years ago

FerrisLee-180505 commented 4 years ago

Do you want to request a feature or report a bug?

I think is a bug

What is the current behavior?

the order list with error-index when inserting some text into 2 order list.

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem. You can use this jsfiddle to get started: https://jsfiddle.net/gmertk/e61z7nfa/.

the demo link: http://www.roserobertson.me/markdown-draft-js/ To Test:

  1. Enter some text between the two OrderLists

there are some videos and screenshots for regression. The following: image

markdown-draft-js

What is the expected behavior?

With correct index

Which versions of Draft.js, and which browser / OS are affected by this issue? Did this work in previous versions of Draft.js?

Chrome Version 79.0.3945.88 (Official B

uild) (32-bit)

Rosey commented 4 years ago

Thanks very much for this report! I'll take a look when I have a chance 😊

alibosworth commented 4 years ago

Also running into this on a project called Flow, it appears to have been solved here, though no coverage has been added for it. I'll look at adding that.

Rosey commented 4 years ago

Merged into main, still need to publish a new release with the fix