Skip to content

Correctly extract titles with multiple line breaks#1874

Open
comkieffer wants to merge 1 commit intojrnl-org:developfrom
comkieffer:correctly-handle-entries-starting-with-linebreaks
Open

Correctly extract titles with multiple line breaks#1874
comkieffer wants to merge 1 commit intojrnl-org:developfrom
comkieffer:correctly-handle-entries-starting-with-linebreaks

Conversation

@comkieffer
Copy link
Copy Markdown

Closes #1676 (Starting an entry with multiple line breaks splits the end of the first line by the number of line breaks)

Since the location of the separator was computed on the text.lstrip() but the function returned the substring of text the title would be cut off.

Checklist

  • I have read the contributing doc.
  • I have included a link to the relevant issue number.
  • I have checked to ensure there aren't other open pull requests
    for the same issue.
  • I have written new tests for these changes, as needed.

Since the location of the separator was computed on the `text.lstrip()`
but the function returned the substring of `text` the title would be cut
off.
@comkieffer comkieffer force-pushed the correctly-handle-entries-starting-with-linebreaks branch from 5f4bcbf to 070350f Compare May 25, 2024 17:24
@comkieffer
Copy link
Copy Markdown
Author

I've rebased this onto the latest develop/. Let's see if the tests pass.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Starting an entry with multiple line breaks splits the end of the first line by the number of line breaks

1 participant