Skip to content
This repository has been archived by the owner on Sep 18, 2021. It is now read-only.

Indent long paragraphs #22

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Indent long paragraphs #22

wants to merge 2 commits into from

Conversation

melsaa
Copy link

@melsaa melsaa commented Oct 25, 2020

Fix #5

@gridhead gridhead self-requested a review October 29, 2020 12:38
@gridhead gridhead self-assigned this Oct 29, 2020
@gridhead
Copy link
Member

Thanks. I'll get into reviewing this PR shortly.

Copy link
Member

@gridhead gridhead left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks strange.

image

You are requested to make changes to ensure consistent indentations.

@melsaa
Copy link
Author

melsaa commented Oct 29, 2020

I'm sorry I'm not sure if I understand correctly, could you tell me, from the above screenshot, which part should be indented and which part shouldn't?

@gridhead
Copy link
Member

@melsaa, the first line of every paragraph should also follow the indentation specified. If that is done, it would look more uniform.

@melsaa
Copy link
Author

melsaa commented Nov 1, 2020

I fixed it, now each paragraph is indented properly.

Copy link
Member

@gridhead gridhead left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The newline apparently messes up the indentation again and resets it back to the prompt's leftmost end.

Here, take a look.

image

@gridhead
Copy link
Member

gridhead commented Nov 4, 2020

Notice how 2015 and 2016 are rendered correctly, while 2017 is messed up.

image

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Suggest a per-newline indentation model for long texts
2 participants