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

Feature Request: Add option for adding side margins #350

Open
lukew3 opened this issue Dec 27, 2021 · 2 comments
Open

Feature Request: Add option for adding side margins #350

lukew3 opened this issue Dec 27, 2021 · 2 comments

Comments

@lukew3
Copy link

lukew3 commented Dec 27, 2021

It's uncomfortable to have the text reach all the way to the edge of the screen. I would suggest allowing the user to use a -m option with an optional value to set the side margins. Could default to 10px if the flag is used but be set to any other quantity of pixels.

@Phlip
Copy link

Phlip commented Mar 22, 2022

I have the opposite problem. When I use the web server and 'wget' a page, it has no margins, which is what I want. (The README must appear in a tablet.) But when I use --export, the page has margins. So we both need a toggle to explicitly declare when we need margins. (also a table of contents wouldn't suck)

@Phlip
Copy link

Phlip commented Mar 22, 2022

I'm good. --no-inline moves the CSS to files that my browser can't read; then I use string surgery to remove the lines that say <link rel="stylesheet" href="/__/grip/asset/dark_colorblind-3d9d29921e03.css" />. The result is clean HTML with no margins

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

No branches or pull requests

2 participants