nextcloud / nextcloud/notes-android
Vertical spacing incorrect for nested lists and lists with multi-paragraph items
Nobody has claimed this yet.
- 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).
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.
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
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- 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