Skip to content
This repository has been archived by the owner on Apr 14, 2022. It is now read-only.

⛏ use system print function #2

Closed
wants to merge 1 commit into from
Closed

Conversation

chafel
Copy link

@chafel chafel commented Jan 1, 2017

看到你在 v2ex 这个讨论 上说会调用浏览器打印方法,但实际上你放的是个链接,这个 pr 里调用了 window.print 方法,而且用 id 指定了打印的内容,虽然我看你 css 上已经针对 print 做 media query 了,但如果页面里还有没有匹配到的元素还得添加样式规则

@neruthes
Copy link
Owner

I recommend that one shall always produce PDF on his/her own and this may remain the default behavior. Consider that a visitor may simply use Command-P when he/she would like to print or create his/her own PDF; then the pre-produced PDF can be a good complementary which allows the carefully designed visual details (e.g. typography) to be preserved.

Additionally, please use English instead for a better international communication environment; nonetheless, I appreciate your kindness.

@neruthes neruthes closed this Jun 13, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants