Skip to content

Feedback from customer on PDF#1065

Open
ranjinimn wants to merge 1 commit intosuse-edge:mainfrom
ranjinimn:PDF-customer-feedback
Open

Feedback from customer on PDF#1065
ranjinimn wants to merge 1 commit intosuse-edge:mainfrom
ranjinimn:PDF-customer-feedback

Conversation

@ranjinimn
Copy link
Collaborator

@ranjinimn ranjinimn commented Feb 24, 2026

Hi, I have accessed the PDF version of the Edge documentation, and there are some issues at least for the last few versions of the document. For example you can take a look at https://documentation.suse.com/suse-edge/3.5/pdf/edge_en.pdf:

Images that represent diagrams aren't correct, they are missing all the text, this text is displayed correctly on the HTML page, like PDF pages 21 (xxi), 22 (xxii), 24 (xxiv), 31 (3), 55 (27).
Text that represents code goes beyond document right margin, like in pages 76 (48), 80 (52), 82 (54), 170 (142), 174 (146), 176 (148). And sometimes beyond the bottom margin and overlap the page number like pages 199 (171), 200 (172), 202 (174), 203 (175).
Screenshots are larger than page size and go beyond the right margin or the bottom margin and you cannot really see the whole screenshot that is necessary, like in pages 178 (150), 180 (152).
The pages I mention are only examples. I guess the whole PDF should be reviewed in detail since there can be more issues.
I hope it helps.

https://suse.slack.com/archives/C072TF88KNU/p1771545274219749

Signed-off-by: Ranjini M N <ranjini.n@suse.com>
@ranjinimn ranjinimn self-assigned this Feb 24, 2026
@ranjinimn ranjinimn requested a review from hardys February 24, 2026 19:21
Comment on lines +214 to +218
userDataBase64: I2Nsb3VkLWNvbmZpZwpkaXNhYmxlX3Jvb3Q6IGZhbHNlCnNzaF9wd2F1dGg6IFRyd
WUKdXNlcnM6CiAgLSBkZWZhdWx0CiAgLSBuYW1lOiBzdXNlCiAgICBncm91cHM6IH
N1ZG8KICAgIHNoZWxsOiAvYmluL2Jhc2gKICAgIHN1ZG86ICBBTEw9KEFMTCkgTk9
QQVNTV0Q6QUxMCiAgICBsb2NrX3Bhc3N3ZDogRmFsc2UKICAgIHBsYWluX3RleHRf
cGFzc3dkOiAnc3VzZScK
Copy link
Collaborator Author

@ranjinimn ranjinimn Feb 24, 2026

Choose a reason for hiding this comment

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

@hardys This renders differently in the PDF version and runs off the page. Is there an option to replace this with anything more generic?

Copy link
Collaborator

Choose a reason for hiding this comment

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

This, and the strings below, are examples if I understand it correctly. I suggest to shorten them to something like
I2Nsb3VkLW[...]Anc3VzZScK
OK with you @hardys ?

[,console]
----
$6$G392FCbxVgnLaFw1$Ujt00mdpJ3tDHxEg1snBU3GjujQf6f8kvopu7jiCBIhRbRvMmKUqwcmXAKggaSSKeUUOEtCP3ZUoZQY7zTXnC1
$6$G392FCbxVgnLaFw$Ujt00mdpJ3tDHxEg1snBU3GjujQf6f8kvopu7jiCBIhRbRvMmKUqwcmXAKggaSSKeUUOEtCP3ZUoZQY7zTXnC1
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I had to remove the number from the password as it was not breaking the text correctly in the PDF rendition. Please let me know if it's okay.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Again, an example string

== High-level architecture

image::quickstart-elemental-architecture.svg[scaledwidth=100%]
image::quickstart-elemental-architecture.png[scaledwidth=100%]
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We need to use PNG images in the document. I have converted all the SVG images to PNG images as the text in an SVG image was not visible in the PDF version.

Copy link
Collaborator

@fsundermeyer fsundermeyer Mar 6, 2026

Choose a reason for hiding this comment

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

Please do not convert SVGs to PNG. Just leave the SVFs as sources, but use the .png suffix in ASCIIDoc (like in the example above). DAPS will detect that an SVG with the same basename exists and will automatically convert it to PNG.

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.

2 participants