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

Tab character no longer working for nested lists #376

Closed
kenlyon opened this issue Jun 9, 2016 · 7 comments
Closed

Tab character no longer working for nested lists #376

kenlyon opened this issue Jun 9, 2016 · 7 comments
Milestone

Comments

@kenlyon
Copy link

kenlyon commented Jun 9, 2016

Steps to reproduce:

  1. Download and install daux.io.
  2. Open the "docs/00_Getting_Started.md" file in a text editor.
  3. Add a tab character (not four spaces) to the start of any bullet point
    except the first one in a list.
  4. Launch the website.
  5. Click on the View Documentation button to view the "Getting Started" page.

Expected result:

The edited line should be indented one level.

Actual result:

The edited line is appended to the end of the previous line, with the *
character rendered literally, not as a bullet.

Additional information:

This was introduced by commit 9a615ea.

Is this an intended deprecation of tabs or was this behaviour introduced by
accident? I realize tabs aren't officially supported in markdown, but our team
also edits in gitlab, which does support tabs. It'd be great if tabs were still
accepted here.

My environment:

  • Windows 10 Pro (64-bit)
  • PHP v5.4.45
  • node.js v4.3.1
  • grunt-cli v2.14.12
@onigoetz
Copy link
Collaborator

Thanks for your report, I will look into this, the commit you are referring to follows an update to commonmark 0.13.2 from 0.13.0 which contains the following PR : thephpleague/commonmark#234

This seems to change the behaviour with tabs.

Can you please provide me the example as a gist so I can make further tests with it ?

@kenlyon
Copy link
Author

kenlyon commented Jun 13, 2016

I've attached a basic example. The tab character was previously treated the same as four spaces, allowing sub list items to be defined as demonstrated in the attachment. In the same example, I also included a sub list item defined with four spaces to allow a comparison.

example.txt

@onigoetz
Copy link
Collaborator

I opened an issue directly in the commonmark project about this : thephpleague/commonmark#258 We'll see what they can do about it

@onigoetz onigoetz added this to the 1.0 milestone Aug 12, 2016
@onigoetz
Copy link
Collaborator

Okay so this error is fixed in commonmark, I will close this once the fix is released.

@onigoetz
Copy link
Collaborator

onigoetz commented Sep 22, 2016

league/commonmark 0.15.0 was released with the fix, but because of webuni/commonmark-table-extension we can't update just yet.

I created the following pull requests to them:

webuni/commonmark-twig-renderer#3
thephpleague/commonmark-ext-table#5
webuni/commonmark-attributes-extension#5

@hason
Copy link

hason commented Sep 26, 2016

@onigoetz I published a new version of webuni/commonmark-table-extension.

@onigoetz
Copy link
Collaborator

Awesome, thanks @hason

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants