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

[Bug] Advanced Markdown Editor on Android #528

Open
raoul-van-oosten opened this issue Apr 23, 2019 · 11 comments

Comments

Projects
None yet
2 participants
@raoul-van-oosten
Copy link

commented Apr 23, 2019

I really like this extension, but on Android it does not work at all. Every time I insert a space or move the cursor via the touch screen, text already in the note is being duplicated. It is also not possible to select text.

@mobitar

This comment has been minimized.

Copy link
Member

commented Apr 23, 2019

Hmm this was an issue that was fixed several months ago. Is it possible you're running a cached version? Also, what version of the app are you using, and what version of Android?

@raoul-van-oosten

This comment has been minimized.

Copy link
Author

commented Apr 23, 2019

App version is 3.0.5 and Android version 9 (Samsung Galaxy S9, it doesn't say much more than that, but it is the latest).

I logged in into the app again, and the problem persists.

@mobitar

This comment has been minimized.

Copy link
Member

commented Apr 23, 2019

Can you go into your Android Settings > Storage > S. Notes > Clear Cache, then try launching the app again and seeing if the issue persists?

@raoul-van-oosten

This comment has been minimized.

Copy link
Author

commented Apr 24, 2019

Clearing the cache did not solve it, but it works much better after rebooting and reinstalling. Thanks!

The only quirk that is still apparent is that pressing enter move the cursor to the beginning of the line and moves the line down, rather than inserting a new line below (cf. vim, it does Shift-O rather than O).

@mobitar

This comment has been minimized.

Copy link
Member

commented Apr 24, 2019

Any chance you can capture a screen recording of that happening?

@raoul-van-oosten

This comment has been minimized.

Copy link
Author

commented Apr 29, 2019

Sorry for taking so long, but here is a screen capture.

https://youtu.be/xZN4f8NecaE

Typing is alright most of the time, but it seems when I insert a new line after typing a single word on a line, the line is inserted above.

@mobitar

This comment has been minimized.

Copy link
Member

commented Apr 29, 2019

Thanks for the video. Although, I'm still having a hard time seeing what you mean. So when you press enter in the middle of a line, the expected behavior is that anything to the right of the cursor moves to the line below, and the cursor is now at the beginning of this new line. It appears this is the behavior from the video. Am I overlooking something?

@raoul-van-oosten

This comment has been minimized.

Copy link
Author

commented Apr 29, 2019

I should have given a better explanation. In the video, the problem arises as 0:17. All enters before that work as they should. So it seems the problem only occurs when I press enter after entering a single word on a line, not when I split a line that has been created earlier.

@mobitar

This comment has been minimized.

Copy link
Member

commented Apr 30, 2019

Oh I see, it inserts a newline above. Hmm. I'll have to look into this. Only on Android I'm assuming? This is going to be a tricky one to fix, given we don't have control over the inner rendering of the editor.

@raoul-van-oosten

This comment has been minimized.

Copy link
Author

commented Apr 30, 2019

I only use Android, so no clue what happens on iPhones. It would be great if you could look into it, but to me it is only a minor nuisance, so I am happily using the extension anyway.

@mobitar

This comment has been minimized.

Copy link
Member

commented Apr 30, 2019

Btw, this is the main open-source repository for the editor, which we integrate: https://github.com/Ionaru/easy-markdown-editor

So it may be this issue was fixed in there but we haven't pulled in the changes. If not, it may not be a bad idea to post this same issue there. There was previously an issue where spellcheck on Android would cause duplication, and it was fixed in this repo, and we pulled the changes. That issue may in fact be related to this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.