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

Vertical padding doesn't work for nodes #4855

Open
mi-kra opened this issue Sep 19, 2023 · 3 comments
Open

Vertical padding doesn't work for nodes #4855

mi-kra opened this issue Sep 19, 2023 · 3 comments
Labels
Status: Triage Needs to be verified, categorized, etc Type: Bug / Error Something isn't working or is incorrect

Comments

@mi-kra
Copy link

mi-kra commented Sep 19, 2023

Description

"padding-top" and "padding-bottom" styles doesn't work while "padding-left" and "padding-right" works well.
Furthermore, "padding" style changes left and right only.

Steps to reproduce

Paste sample to LiveEditor and take a look for cell-padding inside nodes.

Screenshots

No response

Code Sample

graph LR
classDef paddingleft  padding-left:50px
classDef paddingtop   padding-top:50px
classDef paddingall   padding-top:50px,padding-bottom:50px,padding-left:50px,padding-right:50px
Padding_Left:::paddingleft
Missing_Padding_Top:::paddingtop
Missing_Padding_Top_And_Bottom:::paddingall

Setup

  • Mermaid version: current, 10.4
  • Browser and Version: Chrome

Suggested Solutions

if it is not a bug, add suport for these styles :)

Additional Context

New fake css style to override/modify global config property "padding" for specified node. I need to reduce padding size for some small nodes.

for example:
classDef paddingtop mermaid-config-padding:5px

@mi-kra mi-kra added Status: Triage Needs to be verified, categorized, etc Type: Bug / Error Something isn't working or is incorrect labels Sep 19, 2023
@NmN03jain
Copy link

so i have to add vertical padding in it ?

@Aryan-Dahiya-23
Copy link

Is the issue still open? I would like to work on this issue.

@mi-kra
Copy link
Author

mi-kra commented Oct 9, 2023

so i have to add vertical padding in it ?

Yes, please :-)

@jgreywolf jgreywolf added roadmap items to add to roadmap for auto workflow and removed roadmap items to add to roadmap for auto workflow labels Nov 16, 2023
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

4 participants