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

Styling messed up for json resume themes #155

Closed
surendra2014 opened this issue Apr 15, 2016 · 10 comments
Closed

Styling messed up for json resume themes #155

surendra2014 opened this issue Apr 15, 2016 · 10 comments

Comments

@surendra2014
Copy link

surendra2014 commented Apr 15, 2016

FRESH: hackmyresume BUILD test_resume.json TO out/rez.all -t compact works fine with styles
JSON : hackmyresume BUILD test_resume.json TO out/rez.all -t themes/jsonresume-theme-kendall
works fine but styles messed up.

I noticed that css files are generated for fresh but not for json resume themes.

@camdenb
Copy link

camdenb commented Sep 5, 2016

Yeah I'm having this issue too. Seems like most themes are made for webpages so they're usually kinda broken in PDF form.

@hacksalot
Copy link
Owner

I'll take a look- thanks for the report.

@adenta
Copy link

adenta commented Feb 3, 2018

Here is what I get when I follow the above.

Everything is super small.

This happens with both FRESH and JSON

rez.pdf

@hacksalot
Copy link
Owner

Thanks @adenta for the sample PDF. That's more of a postage stamp resume. Was this generated through Phantom, wkhtmltopdf, or WeasyPrint?

@hacksalot hacksalot added this to the v2.0.0 milestone Feb 3, 2018
@adenta
Copy link

adenta commented Feb 3, 2018

wkhtmltopdf

Should I try Phantom?

@hacksalot
Copy link
Owner

hacksalot commented Feb 3, 2018

Well, in any case something is broken with wkhtmltopdf or our handling of it. As a sanity check, if you invoke wkhtmltopdf from the command line, against the HTML resume generated by HackMyResume...

wkhtmltopdf resume.pdf.html output.pdf

...would the sizing still be messed up?

@adenta
Copy link

adenta commented Feb 3, 2018

Yes, the sizing is still messed up. Going to try Phantom and will report back.

@adenta
Copy link

adenta commented Feb 3, 2018

I can confirm this is isolated to wkhtmltopdf

https://cl.ly/3F0u0N2Z050F

Looks like phantom is working about as expected.

@hacksalot
Copy link
Owner

hacksalot commented Feb 3, 2018

Thanks. If the sizing is also messed up under CLI wkhtmltopdf then it's possibly a characteristic of the HTML HackMyResume is emitting. It could also be a wkhtmltopdf bug. Even if it is an wkhtmltopdf bug, the goal of HMR is to emit HTML that most HTML-to-PDF generators can chew through. If we're not doing that, it's a bug, or at least an enhancement. I mean there exists some HTML document displaying this resume that wkhtmltopdf will display correctly. Ideally, built-in themes are optimized in this way.

@hacksalot
Copy link
Owner

I can generate a reasonable jsonresume-theme-kendall in the latest dev code with:

git clone https://github.com/LinuxBozo/jsonresume-theme-kendall
npm i
hackmyresume my/resume.json test/out.all -t jsonresume-theme-kendall

Is this drastically different from what you guys are seeing?

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants