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

The option of gantt for the spaces for the section names. #353

Merged
merged 4 commits into from
May 25, 2016

Conversation

tkfm-yamaguchi
Copy link

At first, thank you for the nice library.

According to mermaidAPI.js, sidePadding of the gantt enables to specify the space allocated for the section name to the left of the activities., but the total width of the gantt seems to be changed when sidePadding is changed.

fireshot capture 72 - - http___127 0 0 1_52283_ganttsidespan html

This PR enables to specify the spaces for the section name without affecting the total with of the gantt.

fireshot capture 71 - - http___127 0 0 1_52283_ganttsidespan html

There is one thing to note.

I change the option name from sidePadding to leftPadding because the leftPadding means only the left side of spaces for the section names in contrast with that sidePadding means the both side of spaces on the gantt.

I don't particular about the name issue above, so I'm grad to hear your opinion.

Thanks.

@knsv
Copy link
Collaborator

knsv commented May 25, 2016

Thanks, good stuff!

@knsv knsv merged commit 0ab0be0 into mermaid-js:master May 25, 2016
@tkfm-yamaguchi
Copy link
Author

Thanks for merging 😄

@tkfm-yamaguchi tkfm-yamaguchi deleted the gantt-width branch May 27, 2016 03:03
@knsv
Copy link
Collaborator

knsv commented May 29, 2016

This is now part of release 6.0.0. Thx again.

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.

None yet

3 participants