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

User guide, shortcodes, print preview: fix broken links #1449

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

deining
Copy link
Collaborator

@deining deining commented Feb 28, 2023

User guide, page Shortcodes, regular view:

grafik

User guide, page Shortcodes, print view:

grafik

The link to the flag is broken in the print view. This PR address this issue and fixes it by using absolute image links.

@deining deining added the user guide Improvements to user guide label Feb 28, 2023
@deining deining force-pushed the output-format-print-broken-links branch from 56de278 to 02f7e65 Compare February 28, 2023 15:08
Copy link
Collaborator

@chalin chalin left a comment

Choose a reason for hiding this comment

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

Hmm, maybe the source of the problem is with the print view?

@deining
Copy link
Collaborator Author

deining commented Feb 28, 2023

Hmm, maybe the source of the problem is with the print view?

Yes, I consider this PR a workaround only.

@deining
Copy link
Collaborator Author

deining commented Feb 28, 2023

These link check results reveal that there definitely seems to be a problem with broken links with our print preview.

@deining deining force-pushed the output-format-print-broken-links branch from 02f7e65 to 440b47d Compare June 11, 2023 09:24
@chalin chalin modified the milestones: 23Q3, 23Q4 Jul 27, 2023
@chalin chalin modified the milestones: 23Q4, 24Q1 Nov 3, 2023
@chalin chalin modified the milestones: 24Q1, 24Q2 Jan 11, 2024
@chalin chalin modified the milestones: 24Q2, 24Q3 Apr 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
user guide Improvements to user guide
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants