Skip to content
This repository was archived by the owner on Nov 1, 2017. It is now read-only.

Conversation

@raganwald
Copy link
Contributor

Task lists within block quotes did not work correctly because the regular expression used to match an item with its place in the GFM form did not recognize items within blockquotes.

These changes close #31 by updating itemPattern to recognize a task list item nested within an arbitrary level of block quoting.

📣 @github/js @mtodd

@raganwald
Copy link
Contributor Author

  • Add tests for nested block quotes
  • Split regex for readability
  • All update tests in one file

Copy link
Contributor

Choose a reason for hiding this comment

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

I think that should be \s*. >>> works to create a three-deep blockquote.

@raganwald
Copy link
Contributor Author

@mtodd:

  1. Are we good to go?
  2. Can we walk through deployment on this together?

Copy link
Member

Choose a reason for hiding this comment

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

Technically this stanza should be optional prefix; and item indicator would need to be clarified as list item indicator.

Copy link
Member

Choose a reason for hiding this comment

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

Regexp looks solid! Thanks for the input, @aroben!

Copy link
Member

Choose a reason for hiding this comment

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

👍 looks good.

mtodd added a commit that referenced this pull request Sep 16, 2014
Support task lists within block quotes
@mtodd mtodd merged commit 66ddf95 into master Sep 16, 2014
@mtodd mtodd deleted the 31-quoted-task-lists branch September 16, 2014 00:02
@raganwald
Copy link
Contributor Author

🎉

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.

Issues with task lists in blockquotes

4 participants