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

Prettified HTML tables render with erroneous top blank space, whereas compacted do not. #5328

Open
RokeJulianLockhart opened this issue Feb 26, 2024 · 0 comments
Labels
Status: Triage Needs to be verified, categorized, etc Type: Bug / Error Something isn't working or is incorrect

Comments

@RokeJulianLockhart
Copy link

RokeJulianLockhart commented Feb 26, 2024

Description

The undermentioned code, solely when multiline/indented, creates a table with an absurdly padded top margin.

Steps to reproduce

  1. https://mermaid.live/edit#pako:eNp1kDEPgjAQhf8KuZnEuDaEwTA6IZt1OOghRFpMOWII4b_bEhFJcHtfX_ve9UYoWkUgoGzaV1Gh5SBLpJFsjlcJXkiOGPOG4gUqQvUBj_arZzPOqOPsHB2c3DHSjeFgee7P1-CI81YNf1vUHHbyLWrPSDfGtuUn2NH6M7hBCJqsxlq5fYzSBIEEN5QmCcJJhfbhVzK5e9hzexlMAYJtTyH0T4VMSY13ixpEiU1H0xtcP30j

    Screenshots

    Prettified HTML tables render with erroneous top blank space, whereas compacted do not. mjbvz/vscode-markdown-mermaid#245 (comment)

    image

    Code Sample

    flowchart TD
    	n1["
    		<table>
    		<thead>
    			<tr>
    				<th>TestTL</th>
    				<th>TestTR</th>
    			</tr>
    		</thead>
    		<tbody>
    			<tr>
    				<td>TestBL</td>
    				<td>TestBR</td>
    			</tr>
    		</tbody>
    		</table>
    	"]
    
  2. https://mermaid.live/edit#pako:eNpFjrEOgkAMhl-FdDa5uF4Ig2F0QjbrUGgRIneY44ghhHf38FCn_l_y_W0XqAcW0ND0w6tuyfmkzNGit8crQuqp6iVLfSvEYbgtZqWMvjynKsQfFhHVpqivXg087y3-aKetxX8sIu6tqKt4E-EGBzDiDHUc_lvQJglCWG0EQYfI5B4IaNfg0eSHy2xr0N5NcoDpyeQl7-juyIBuqB9lfQMRak_9

    Screenshots

    Prettified HTML tables render with erroneous top blank space, whereas compacted do not. mjbvz/vscode-markdown-mermaid#245 (comment)

    image

    Code Sample

    flowchart TD
    	n1["<table><thead><tr><th>TestTL</th><th>TestTR</th></tr></thead><tbody><tr><td>TestBL</td><td>TestBR</td></tr></tbody></table>"]
    

Setup

Mermaid version

I use https://github.com/mjbvz/vscode-markdown-mermaid/blob/bc6dfff7d69202b62be271f8a1bd34a7df9ab920/README.md#markdown-preview-mermaid-support:~:text=currently%20supports%20mermaid%20version%2010.8.0, which "Currently supports Mermaid version 10.8.0".

Browser and Version

  1. code-insiders-1.87.0-1708582258.el8.src

    Version: 1.87.0-insider
    Commit: a0b90ac5e04ea9a0733b68062253a469994e170e
    Date: 2024-02-22T05:47:39.416Z
    Electron: 27.3.2
    ElectronBuildId: 26836302
    Chromium: 118.0.5993.159
    Node.js: 18.17.1
    V8: 11.8.172.18-electron.0
    OS: Linux x64 6.7.5-1-default

  2. and https://download.opensuse.org/repositories/openSUSE:/Factory/standard/x86_64/MozillaFirefox-122.0.1-1.1.x86_64.rpm

Suggested Solutions

I don't know what the issue is, so I can't provide one.

Additional Context

Reported here per mjbvz/vscode-markdown-mermaid#245 (comment).

@RokeJulianLockhart RokeJulianLockhart added Status: Triage Needs to be verified, categorized, etc Type: Bug / Error Something isn't working or is incorrect labels Feb 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Triage Needs to be verified, categorized, etc Type: Bug / Error Something isn't working or is incorrect
Projects
None yet
Development

No branches or pull requests

1 participant