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

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