Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Improve block quote formatting#168

Merged
dscho merged 1 commit intogitgitgadget:mastergitgitgadget/gitgitgadget:masterfrom
webstech:block-quotewebstech/gitgitgadget:block-quoteCopy head branch name to clipboard
Dec 13, 2019
Merged

Improve block quote formatting#168
dscho merged 1 commit intogitgitgadget:mastergitgitgadget/gitgitgadget:masterfrom
webstech:block-quotewebstech/gitgitgadget:block-quoteCopy head branch name to clipboard

Conversation

@webstech
Copy link
Contributor

The fromString function from "html-to-text" does "lazy"
quoting with only a leading > for the quote. This makes
cover letters that include quoting hard to read.

This change adds a blockquote formatter that will put > at
the start of each line of the quote. Quoting inside of quotes
will only add a >. Not perfect but better than relying on
mail formatters to handle lazy quoting.

Signed-off-by: Chris. Webster chris@webstech.net

@webstech webstech requested a review from dscho December 12, 2019 00:54
dscho
dscho previously approved these changes Dec 13, 2019
lib/markdown-renderer.ts Outdated Show resolved Hide resolved
The fromString function from "html-to-text" does "lazy"
quoting with only a leading > for the quote.  This makes
cover letters that include quoting hard to read.

This change adds a blockquote formatter that will put `> ` at
the start of each line of the quote.  Quoting inside of quotes
will only add a `>`.  Not perfect but better than relying on
mail formatters to handle lazy quoting.

Signed-off-by: Chris. Webster <chris@webstech.net>
@webstech
Copy link
Contributor Author

This is a fix for issue #161.

@dscho dscho merged commit a143b6c into gitgitgadget:master Dec 13, 2019
@dscho
Copy link
Member

dscho commented Dec 13, 2019

Very nice, thank you!

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.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.