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

#422 Rxmarkdown update #427

Merged
merged 7 commits into from
Aug 24, 2018
Merged

#422 Rxmarkdown update #427

merged 7 commits into from
Aug 24, 2018

Conversation

stefan-niedermann
Copy link
Member

@stefan-niedermann stefan-niedermann commented Jul 9, 2018

Fixes #422, #287 and #157
This is a basic implementation of v0.1.2 of RxMarkdown using the Reactive X way.

It also implements toggling checkboxes from within the view mode.

problems left:

  • We have to wait for v0.1.3 because of Gives error in List yydcdut/RxMarkdown#50
  • After toggling a checkbox the content is displayed as plain text and does not get processed anymore (unti switching to edit mode and back to view mode)
  • toggling the last checkbox (last line) causes the last letter of this line to disappear. Not sure yet if this is a bug of RxMarkdown or a result of wrong usage.

Checkbox support is out of scope because it still causes errors.

I did not have any success implementing this without Reactive X (the checkbox-listener was not called - maybe a bug, too).

@stefan-niedermann
Copy link
Member Author

v0.1.3 has been released https://github.com/yydcdut/RxMarkdown/releases/tag/v0.1.3

yydcdut/RxMarkdown#50 should be fixed in this release. As soon as i have some time i will update and see how to fix the checkbox-issues.

Remove support of toggling checkboxes due to bugs
@stefan-niedermann
Copy link
Member Author

Sorry to disappoint you, but i removed the support for toggling checkboxes for now because it still causes errors.

I will open a new issue to track it since it basically is possible now. But to move on, we should first upgrade without adding more features.

stefan-niedermann and others added 2 commits August 24, 2018 14:22
Removes the auto-linking feature since it has been implemented upstream
@stefan-niedermann stefan-niedermann merged commit 1488704 into master Aug 24, 2018
@stefan-niedermann stefan-niedermann deleted the rxmarkdown-update branch August 24, 2018 12:23
@stefan-niedermann stefan-niedermann added this to the 0.21.0 milestone Aug 24, 2018
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

Successfully merging this pull request may close these issues.

None yet

1 participant