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

Is it possible to adjust the height of a rendered markmap? #1144

Open
ajfriesen opened this issue Aug 3, 2022 · 6 comments · May be fixed by #1365
Open

Is it possible to adjust the height of a rendered markmap? #1144

ajfriesen opened this issue Aug 3, 2022 · 6 comments · May be fixed by #1365

Comments

@ajfriesen
Copy link

When using MarkMaps they are always a fixed size.
We have huge whitespace below the created mindmap svg.
Especially when you create markmaps with a lot of horizontal content which could easily be shown on the page. But the whitespace is not allowing that.

Is it somehow possible to increase the height of the rendered markmap?

@LisaFC
Copy link
Collaborator

LisaFC commented Aug 3, 2022

@llhhbc you added the MarkMap integration, any ideas? I can take a look too.

@LisaFC
Copy link
Collaborator

LisaFC commented Aug 11, 2022

Have just seen that the height is hardcoded here:
https://github.com/google/docsy/blob/main/layouts/partials/scripts.html#L16

I need to fix something else in that file, and then I'll see if I can make it more flexible.

@ajfriesen
Copy link
Author

Any update on this?

@emckean
Copy link
Collaborator

emckean commented Sep 2, 2022

ugh, sorry, I took over the fix but didn't see this issue. The other fix is in #1175, but I'm also checking on something else so I'll keep this in mind now!

Short-term, I think you could change that height in the /layouts/partials/scripts.html file

@blake-glyd
Copy link

Any updates on making this more flexible?

@deining deining linked a pull request Jan 24, 2023 that will close this issue
@llhhbc
Copy link
Contributor

llhhbc commented Mar 1, 2023

@LisaFC Sorry, the previous level was limited, and it could only be integrated, but it could not be extended.

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 a pull request may close this issue.

5 participants