Skip to content

Conversation

@novoselt
Copy link
Collaborator

@novoselt novoselt commented Dec 4, 2025

Rendered view:
image
VPAT is now served as a PDF from the official template - it is likely to be more convenient for customers bookkeeping and discussions. The link is somewhat "lost" on the new page, but those who are interested in it really should read the page first.

Comment on lines +56 to +58
<a href="/documents/SageMathInc_VPAT2.5Rev_WCAG_February2025_December2025.pdf">
Voluntary Product Accessibility Template, VPAT®
</a>{" "}
Copy link
Contributor

Choose a reason for hiding this comment

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

my gut feeling is this should be an uppercase <A ...> tag, because they have a prefix logic. The path should be relative as well (just remove the leading "/"). I'm not sure, though, maybe with PDF something isn't ok with that approach.

Copy link
Contributor

Choose a reason for hiding this comment

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

but I see, you probably need the leading /

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

If I remove / it resolves to policies/documents instead of just documents, so I would need to move the file then - is this preferable?

As for a vs A codex suggests it should be lower case for the build-in HTML element rather than capital A for the component. I don't find it very convincing, but if I add the necessary import for A and use it, the page renders fine, but the PDF does not open, giving an error instead:
image

@haraldschilly haraldschilly added PR-TODO-cocalc2 merge/migrate this PR into CoCalc2 in the future PR-positive review ready to merge and removed PR-needs work labels Dec 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR-positive review ready to merge PR-TODO-cocalc2 merge/migrate this PR into CoCalc2 in the future

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants