swinton / pdfify

Generates Octocat-friendly PDFs from Markdown
MIT License
3 stars 1 forks source link

Small Text in PDF #1

Closed gracepark closed 7 years ago

gracepark commented 7 years ago

Hi there! It looks like the text when in the PDF is fairly tiny. I've attached the .md and .pdf file.

testing_pdfify.pdf

This was the markdown:

## Testing pdfify

### TEST
**This is so cool**
  - Thanks **Steve**!

### New UPDATE
  - This is working now!

I had an original issue with installing wkhtmltopdf, but I was able to fix that by updating the lastest versions of Homebrew, Homebrew-Cask, and all Taps.

The version of wkhtmltopdf I have is{ wkhtmltopdf 0.12.4 (with patched qt)

Let me know if there's anything I can test to help out! <3

swinton commented 7 years ago

Thanks for the bug report, @gracepark.

Definitely something odd going on here. This is the output I'm getting from that same Markdown:

@gracepark, when you run pdfify, you should see a line of output like:

/var/folders/_q/4594nnj94wb7kll_9hl7tk340000gn/T/tmp.RJ0vJUow/XXXX.html

Could you grab that HTML file and upload here? :pray:

Thank you :bow:

gracepark commented 7 years ago

@swinton - Yes! I got this: /var/folders/bg/73v8k2k101q3tv2m8kb53k3w0000gn/T/tmp.MTORqafg/testing_pdfify.html

swinton commented 7 years ago

/var/folders/bg/73v8k2k101q3tv2m8kb53k3w0000gn/T/tmp.MTORqafg/testing_pdfify.html

:+1:

Could you copy the contents of that file here? I.e.

cat /var/folders/bg/73v8k2k101q3tv2m8kb53k3w0000gn/T/tmp.MTORqafg/testing_pdfify.html | pbcopy

and paste here 👍

gracepark commented 7 years ago
<!DOCTYPE html>
<html>
<head>
  <meta charset="utf-8">
  <meta name="generator" content="pandoc">
  <meta name="viewport" content="width=device-width, initial-scale=1.0, user-scalable=yes">
  <title></title>
  <style type="text/css">code{white-space: pre;}</style>
  <!--[if lt IE 9]>
    <script src="//cdnjs.cloudflare.com/ajax/libs/html5shiv/3.7.3/html5shiv-printshiv.min.js"></script>
  <![endif]-->
</head>
<body>
<h1 class="letterhead">
  <svg height="42" width="42" class="octicon octicon-mark-github" viewBox="0 0 16 16" version="1.1" aria-hidden="true">
    <path fill-rule="evenodd" d="M8 0C3.58 0 0 3.58 0 8c0 3.54 2.29 6.53 5.47 7.59.4.07.55-.17.55-.38 0-.19-.01-.82-.01-1.49-2.01.37-2.53-.49-2.69-.94-.09-.23-.48-.94-.82-1.13-.28-.15-.68-.52-.01-.53.63-.01 1.08.58 1.23.82.72 1.21 1.87.87 2.33.66.07-.52.28-.87.51-1.07-1.78-.2-3.64-.89-3.64-3.95 0-.87.31-1.59.82-2.15-.08-.2-.36-1.02.08-2.12 0 0 .67-.21 2.2.82.64-.18 1.32-.27 2-.27.68 0 1.36.09 2 .27 1.53-1.04 2.2-.82 2.2-.82.44 1.1.16 1.92.08 2.12.51.56.82 1.27.82 2.15 0 3.07-1.87 3.75-3.65 3.95.29.25.54.73.54 1.48 0 1.07-.01 1.93-.01 2.2 0 .21.15.46.55.38A8.013 8.013 0 0 0 16 8c0-4.42-3.58-8-8-8z">
    </path>
  </svg>
  <span><em>GitHub</em> Professional Services</span>
</h1>
<h2 id="testing-pdfify">Testing pdfify</h2>
<h3 id="test">TEST</h3>
<p><strong>This is so cool</strong></p>
<ul>
<li>Thanks <strong>Steve</strong>!</li>
</ul>
<h3 id="new-update">New UPDATE</h3>
<ul>
<li>This is working now!</li>
</ul>
</body>
</html>

@swinton ^ Hope that helps!

swinton commented 7 years ago

Some potentially useful things here:

http://blog.gluga.com/2012/05/wkhtmltopdf-font-and-sizing-issues.html

Things to try:

swinton commented 7 years ago

The issue was caused by @gracepark's MAHOOSIVE screen, it seems. :smile:

Running pdfify on a smaller screen rectified. Adding --print-media-type fixed it.

hectorsector commented 7 years ago

@swinton I experienced this issue, and then it didn't happen anymore, but now it's happening again. I took a hammer to your script and tried all the suggestions in the blog post as well as some more I found from a Google search, featuring such diddy's as --dpi XX, --zoom X, and fixing the HTML container width. DPI and Zoom worked, but I imagine that's inconsistent across machines.

swinton commented 7 years ago

@hectorsector could you please make sure you're running the latest version of pdfify, brew upgrade pdfify? 🙏

hectorsector commented 7 years ago

Yessir 😢 Error: swinton/brews/pdfify 0.1.2 already installed

swinton commented 7 years ago

@hectorsector, that's strange.

Could you open a separate issue in this repo?

If you could include the output from the /var/... file, as @gracepark did above, that would be most helpful :bow:

🔖 There's also this (seemingly) related question on StackOverflow that I've been monitoring.

swinton commented 7 years ago

One more question for you, @hectorsector...

What version of wkhtmltopdf are you running (wkhtmltopdf --version)?

I see this issue, that reports font differences between 0.12.3 and 0.12.4. I'm running 0.12.3.

$ wkhtmltopdf --version
wkhtmltopdf 0.12.3 (with patched qt)