Merge lp://staging/~mrasmus/tomdroid/fix-660116 into lp://staging/~tomdroid-maintainers/tomdroid/main
Proposed by
Matthew Rasmus
| Status: | Merged |
|---|---|
| Merged at revision: | 222 |
| Proposed branch: | lp://staging/~mrasmus/tomdroid/fix-660116 |
| Merge into: | lp://staging/~tomdroid-maintainers/tomdroid/main |
| Diff against target: |
40 lines (+6/-4) 2 files modified
res/layout/note_view.xml (+1/-1) src/org/tomdroid/xml/NoteContentHandler.java (+5/-3) |
| To merge this branch: | bzr merge lp://staging/~mrasmus/tomdroid/fix-660116 |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Tomdroid Maintainers | Pending | ||
|
Review via email:
|
|||
Description of the change
The TextView in note_view was ignoring the indentation and bullets when wrapping (it would wrap for "Line 1" instead of " <> Line 1"). Changing its layout_width parameter to FILL_PARENT fixed this. Also, if a note began with more than one listItem's, the first item's bullet point would not appear (which has been fixed as well).
To post a comment you must log in.
Revision history for this message
| Benoit Garret (benoit.garret) wrote | # |
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
Thanks for your work, merged.
Actually, list items at the beginning of a note would have their bullets rendered. The first characters of the note content always are the title and you can't really put lists in there. But the extra check doesn't hurt, so thanks for your eagle eyes.