nextcloud / nextcloud/notes-android

List item (asterisk) is added to the last line

Open
#1,971 0 comments 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

bug Editor::Plain
Dominant language
Java
Stars
1.1k
Forks
168
Avg merge
1d 17h
Merged PRs (30d)
31

Description

Please use GitHub reactions 👍 to show that you are affected by the same issue. Please don't comment if you have no relevant information to add!

Describe the bug
Opening a note containing a list (both ordered and unordered) followed by an empty line automatically adds a list item in place of the last line. If the item isn't manually removed, then it stays as part of the note.

To Reproduce
Steps to reproduce the behavior:

  1. Make a following note via the web UI or otherwise (must include an EOL before EOF):
* foo
* bar

  1. Open the note on Android (Plain edit)
  2. Modify foo to foofoo
  3. The note now contains an extra list item which (added one the note has been opened on Android):
* foofoo
* bar
* 

Expected behavior
The automatically-added list item should not be part of the note. I.e. not included in the text unless user actually adds some text to it.

Screenshots
Screenshot_20231229-134230

Smartphone (please complete the following information):

  • Nextcloud Notes-Version (android app): 4.1.0
  • F-Droid or Play Store: Play Store
  • Android-Version: 13
  • Device: Moto G73

Server

  • Nextcloud version: 28.0.1
  • Nextcloud Notes version (server app): 4.9.1

Stacktrace N/A

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reproducing the issue in the Android app's Plain edit mode using the supplied note content and steps. Trace how a trailing empty list line is handled when the note is opened and edited; done means the temporary empty list item is not persisted unless the user adds text.

Written by the indexing model from the issue text.

Assessment

Tech stack
android, java
Domain
mobile
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.