Skip to content
This repository has been archived by the owner on Mar 17, 2021. It is now read-only.

Add documentation for how to create template PDF files #547

Merged

Conversation

bs-ondem
Copy link
Contributor

Currently, there is only the documentation for the
AttributionDocumentGenerator which describes how to create a template
bundle JAR file. This adds a further documentation for how to create
those template PDF files.

Resolves #546.

Request Reviewer

@sschuberth

Type of Change

documentation

@bs-ondem bs-ondem requested a review from sschuberth June 24, 2020 14:20
used several times which will be list the used dependencies with PURL, file name, license and copyright notes. The
back.pdf file is the last page of the attribution document which is appended without any changes.

After the document is finished, each individual page is exported as the respective template PDF file.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe it's a bit confusing to speak about PDF files before this sentence. The workflow as I understand it is that first do the full design in Word, not knowing / caring that you would export to PDF later. You basically only care about the paper size and the number of pages at this point. And only at the very end I'd explain that you need to export the first page to a PDF file named cover.pdf, the second page to a PDF called copyright.pdf and so on.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I rewrite the section before this sentence without mentioning PDF files.

* back.pdf

The page size is A4 at 72 PPI with 595x842 pixels. The cover.pdf file will be the cover page of the attribution document
which will contain the project name and version starting at 80x547. The copyright.pdf file will be the second page of
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, so the contents of the cover page are filled in at a fixed pixel position? I always thought the pages would need to contain special makers, like $REPLACE_ME_WITH_THE_PROJECT_NAME_AND_VERSION, which get replaced with the contents...

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How do you control the font properties (type, size, style) of the text being inserted?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes all content is fix at a pixel position not only the cover page.

The font type will be loaded from the pdf file, but the size and style are hardcoded.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The font type will be loaded from the pdf file

Ok, so that hardens my assumption that the cause of the font issues we're seeing are caused by not all characters of the font being embedded into the PDF.

but the size and style are hardcoded.

Please add this bit to the docs then.

The page size is A4 at 72 PPI with 595x842 pixels. The cover.pdf file will be the cover page of the attribution document
which will contain the project name and version starting at 80x547. The copyright.pdf file will be the second page of
the attribution document which will contain the project copyright notes starting at 70x659. The content.pdf file will be
used several times which will be list the used dependencies with PURL, file name, license and copyright notes. The
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there also a pixel position for this file?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, it's written on the page at 40,80 until the page bottom margin is reached.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add this bit to the docs then.

@bs-ondem bs-ondem force-pushed the bs-ondem/docs/attribution-document branch from d30a408 to cf7a5be Compare June 25, 2020 12:43
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

warning The version of Java (1.8.0_252) you have used to run this analysis is deprecated and we will stop accepting it from October 2020. Please update to at least Java 11.
Read more here

@sschuberth sschuberth self-requested a review August 10, 2020 08:33
### Basic folder structure


Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: Unrelated code change that should be undone.

Currently, there is only the documentation for the
AttributionDocumentGenerator which describes how to create a template
bundle JAR file. This adds a further documentation for how to create
those template PDF files.

Resolves eclipse-archived#546.

Signed-off-by: Onur Demirci <[email protected]>
@bs-ondem bs-ondem requested a review from sschuberth September 22, 2020 06:36
@bs-ondem bs-ondem force-pushed the bs-ondem/docs/attribution-document branch from cf7a5be to 60eac12 Compare September 22, 2020 07:21
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

warning The version of Java (1.8.0_265) you have used to run this analysis is deprecated and we will stop accepting it from October 2020. Please update to at least Java 11.
Read more here

@bs-ondem bs-ondem merged commit 36c3e6c into eclipse-archived:master Sep 22, 2020
@sschuberth sschuberth deleted the bs-ondem/docs/attribution-document branch September 22, 2020 07:51
@robertklaus-bosch
Copy link

Some remarks from my side.

I think the description by @bs-ondem is a quite exact specification. However I think it is tricky for users to actually create a template based on this. This is the reason, the example bundle exists which also features a basic odt file which can be used.
So I would suggest to keep the specification but not as number one description. Instead .. why not use the example (specification by example if you like)?

And if you want to polish the text a little more, I think it is worth to structure the text block into different things. Starting with a section on what is needed for the bundle (which pages .. what are they for) then a section how to create them (e.g. starting with the example and then a section on the exact specification.

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.

Add documentation for how to create PDF files for the AttributionDocumentGenerator
3 participants