Skip to content

Table ending in empty cell - #357

Closed
drscannell wants to merge 2 commits into
markedjs:masterfrom
drscannell:table-ending-in-empty-cell
Closed

Table ending in empty cell#357
drscannell wants to merge 2 commits into
markedjs:masterfrom
drscannell:table-ending-in-empty-cell

Conversation

@drscannell

Copy link
Copy Markdown

Currently, if the last cell of a table is empty, it will be dropped from the HTML:

| Header 1 | Header 2 |
|----------|----------|
| Cell 1 | Cell 2 |
| Cell 3 | |

This pull request does the following:

  1. Add test for situation described above
  2. Fix bug causing the the last cell to not be added during conversion to HTML.

@rhiokim

Copy link
Copy Markdown

+1

@servant-42

Copy link
Copy Markdown

+1, also reported in #188

servant-42 added a commit to CSNW/marked that referenced this pull request May 13, 2014
@utensil

Copy link
Copy Markdown

+1

@benelog

Copy link
Copy Markdown

👍

@MaXFalstein

Copy link
Copy Markdown

Is there any reason why this pull request has not been merged?

@redfast00

Copy link
Copy Markdown

+1, there is another PR for the same bug in #697

@JoshuaKGoldberg

Copy link
Copy Markdown

Ping @chjj

@joshbruce

Copy link
Copy Markdown
Member

#980

@joshbrucejoshbruce added this to the 0.4.0 - No known defects milestone Dec 25, 2017
@joshbrucejoshbruce removed this from the 0.4.0 - No known defects milestone Apr 4, 2018
@styfle

Copy link
Copy Markdown
Member

Thanks for the PR but this is already fixed.
See the demo to see it working in the latest version of marked.

@styflestyfle closed this Jul 10, 2018
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants

@drscannell@rhiokim@servant-42@utensil@benelog@MaXFalstein@redfast00@JoshuaKGoldberg@joshbruce@styfle