Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add table support #9

Merged
merged 5 commits into from
Dec 8, 2016
Merged

add table support #9

merged 5 commits into from
Dec 8, 2016

Conversation

GHolk
Copy link
Contributor

@GHolk GHolk commented Dec 8, 2016

only gfm table support.
I do not read all code, so maybe some bug. Fixes: #8.

TODO:

  • enable/disable of table border
  • enable/disable of horizantel line after table header
  • enable/disable of bold font face at table header

gholk added 3 commits December 8, 2016 16:57
only gfm table support,
I don't read all your source code,
but it work.
a table in tbl macro look like:

    .TS

    \" default tab symbol is \t,
    \" you can use tab() command change it.
    \" `;` is needed to end tbl command.
    tab(|);

    \" tell tbl how many column do table have.
    l l l l.

    \" table content.

    c11 | c12 | c13 | c14
    c21 | c22 | c23 | c24

    .TE
use table border, not cell border or row border.
add horizantel line after table header.
@kapouer
Copy link
Owner

kapouer commented Dec 8, 2016

Thank you, could you add a test/md/table.md example ? That'll allow me to convert it into a regression test.

gholk added 2 commits December 8, 2016 21:11
deal inline style in table.

use text block to auto wrap text in cell:

        T{
        long text...
        will be wrap.
        ...
        }T
 1. large table
 2. long text table
 3. thing table (this format ugly...)
@GHolk
Copy link
Contributor Author

GHolk commented Dec 8, 2016

ok,

  • there test table
  • rewrite and add some function.

@kapouer kapouer merged commit dfa728c into kapouer:master Dec 8, 2016
@kapouer
Copy link
Owner

kapouer commented Dec 8, 2016

Thank you, i'll have a more thorough look at it later today, and will publish asap.

@GHolk
Copy link
Contributor Author

GHolk commented Dec 8, 2016

about table in troff:

  • man ascii: ascii code table manual.
  • man tbl: troff table macro man page.

@kapouer
Copy link
Owner

kapouer commented Dec 8, 2016

Thanks i had troubles finding this documentation :)

iarna added a commit to npm/npm that referenced this pull request Dec 15, 2016
Add table support.

Credit: @GHolk
PR-URL: kapouer/marked-man#9
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