search

c3er / mdview

Standalone Markdown viewer
MIT License
82 stars 7 forks source link

Application does not render this table. #48

Open ElektroStudios opened 7 months ago

ElektroStudios commented 7 months ago

I don't know why, but the application is not rendering this table:

|Included:|Excluded:|
|:--|:--|
|Released Games|Cancelled or Not yet released Games
|[Compilations](https://gamefaqs.gamespot.com/cpc/category/233-miscellaneous-compilation)|

Maybe the parser fails to recognize the table due the blank white spaces at the start of each line?. I don't know if that could be the issue, but I can say that table renders fine with web-based things like when using the typical web-extensions in the web-browser to read markdown files, or when using other stand-alone applications like Markdown Monster or Sublime Text (Markdown​Live​Preview plugin), and in this GitHub comment:

  • The following table of platform exclusive games was generated programmatically by scraping content from Gamefaqs website:

    Included: Excluded:
    Released Games Cancelled or Not yet released Games
    Compilations

I attach the original markdown file here:

c3er commented 7 months ago

Thanks for filing this issue!

I could reproduce this bug and interestingly, the table renders just fine, if the table is not indented or indented just the right way, i.e. it starts at the same column as the text of the last bullet point.

Screenshot of different table indentations and their renderings

I attached a file with all variants: table-tests.md

I'll take a look as soon as I have time and energy.

It could be that the bug originates from a dependency of the Markdown renderer, e.g. markdown-it-multimd-table. If this is the case, I'll create a bug report at the appropriate project and hope the best.

And, while there is a workaround available (IMO), it is clearly a bug of my tool, because other renderers don't have this problem.

c3er commented 4 months ago

It is actually an issue with dependencies.

I created an issue in markdown-it for one case. The other case is caused by markdown-it-multimd-table and there exists already an issue for exactly this case.

I'll concentrate now on the zoom issue.

c3er commented 2 months ago

I added a ""help wanted" tag:

  1. This issue originates from the dependencies markdown-it and markdown-it-multimd-table. Maybe, there will be someone, who has experience with markdown-it and is interested to fix it.
  2. I want to concentrate on the major restructuring to tackle some other issues.