Skip to content

Conversation

@jleandroperez
Copy link
Contributor

Details:

In this PR we're adding several Paragraph Analysis helpers, and switching the mechanism that removes Styles, from relying on Lines to Paragraphs instead.

Fixes #715
Needs Review: @diegoreymendez

Thanks in advance!

To test:

  1. Launch the Empty Editor
  2. Insert a List
  3. Insert a Blockquote in the first bullet
  4. Type Hello Shift + Enter There
  5. Hit backspace 6 times

Verify that the Blockquote Style does not get removed from the Hello word.

Please note that after hitting backspace 6 times, the caret will be right by the Hello word, but it won't be visible. To be addressed in another PR!

@jleandroperez jleandroperez added this to the Beta 11 (bugfixes) milestone Sep 6, 2017
@jleandroperez jleandroperez self-assigned this Sep 6, 2017
@jleandroperez jleandroperez changed the title Issue/715 not loosing styles anymore Not Loosing Styles! Sep 6, 2017
Copy link
Contributor

@diegoreymendez diegoreymendez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@jleandroperez
Copy link
Contributor Author

Merging after waiting for way over an hour, for Travis to wrap up it's build.

Thanks Diego!

@jleandroperez jleandroperez merged commit 99f41a8 into develop Sep 8, 2017
@jleandroperez jleandroperez deleted the issue/715-not-loosing-styles-anymore branch September 8, 2017 13:22
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.

3 participants