Skip to content

Improve the usability of PDF file#148

Open
tcvan0707 wants to merge 1 commit into
Seneca-ICTOER:mainfrom
tcvan0707:issue-65
Open

Improve the usability of PDF file#148
tcvan0707 wants to merge 1 commit into
Seneca-ICTOER:mainfrom
tcvan0707:issue-65

Conversation

@tcvan0707

@tcvan0707 tcvan0707 commented Dec 9, 2021

Copy link
Copy Markdown

As mentioned in issue #65, I have improved some of the content in this pull request.

I remove the web-only artifacts like "On this page" and "Edit this page", since we do not want it inside our PDF file. web-only artifacts

And another thing is I have adjust some pictures not showing properly in PDF file, for example image not show properly and this is when after adjusting after modifying

On the other hand, some tables are still not fit the PDF page yet, ASCII Collating Sequence for example, and tables under types.md as well. I will file an issue on that, so we can work on it later.

	* Remove artifact On-this-page and Edit-this-page

	* Fixed image does not show properly
@TueeNguyen

Copy link
Copy Markdown
Contributor

Can you drag on drop the pdf here?

@tcvan0707

Copy link
Copy Markdown
Author

Can you drag on drop the pdf here?
Seneca-IPC144.pdf

@TueeNguyen

Copy link
Copy Markdown
Contributor

Can you also change the name of the PR to something more descriptive

@humphd

humphd commented Dec 9, 2021

Copy link
Copy Markdown
Contributor

This is getting better!

I'd love to go further (here or in another PR). Can we do some CSS Print specific styles?

@tcvan0707 tcvan0707 changed the title Fixed issue #65: Improve the usability of PDF file Dec 9, 2021
@tcvan0707

Copy link
Copy Markdown
Author

Can you also change the name of the PR to something more descriptive

I changed it. Thank you.

@TueeNguyen

Copy link
Copy Markdown
Contributor

I skimmed the first article and that potentially could be the answer to some tables not fitting the pdf issue.

@tcvan0707 mr-pdf has @media print { } in its CLI options , you can specify how some of the components to be printed. Take a look at the articles David linked

@humphd

humphd commented Dec 10, 2021

Copy link
Copy Markdown
Contributor

NOTE: I'm happy for some of this to happen in follow up PRs. We don't have to wait for it to be perfect before we merge.

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 this pull request may close these issues.

3 participants