nextcloud / nextcloud/notes-android

Vertical spacing incorrect for nested lists and lists with multi-paragraph items

Open
#3,292 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

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

Description

Description

If a list item comprises multiple paragraphs, or has children, then the vertical spacing between paragraphs or items within the list item should not exceed the vertical spacing between this parent item and other items at the same level.

Consider a Markdown document with the following contents:

Lorem ipsum: Est dolor quia dolorum est illum quasi aperiam autem. Magnam assumenda  hic accusantium dolores et quia culpa officia. Nisi architecto hic  repudiandae

1. One

1. Two

   Two and a half

   - Two and four sixths
   - Two and five sixths

1. Three

Note that some extraneous content (i.e. Lorem ipsum) has been placed at the beginning, in order to work around the design issue reported in #2976, #2983, #3241, and #3246.

The document as rendered by the application has been captured by a screenshot (see below).


Image

Expected behavior

In contrast, a different application (Markor) renders the same document quite differently (see below).

Many other applications render the document similarly, in terms of vertical spacing, in contrast to the rendering by Nexctcloud Notes for Android.

I feel strongly that the vertical spacing rendered by Markor is correct, whereas the spacing unfortunately is not correct as rendered by Nextcloud Notes for Android.

Vertical spacing between list items, at any particular level, should be consistent regardless of any child lists, and also should not exceed the vertical spacing between items at the level of the parent.


Image

Environment

  • Notes Android version: 34.0.0

  • Notes server version: n/a

  • Nextcloud Android version: 34.1.0

  • Nextcloud version: n/a

  • Device: Pixel 4

  • Android Version: 15

  • App Store: F-Droid

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 reported nested-list example in Notes for Android 34.0.0 on the stated Android environment and compare its spacing with the expected Markor rendering. Done means vertical spacing between items at each list level is consistent, including multi-paragraph items and child lists.

Written by the indexing model from the issue text.

Assessment

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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.