-
Notifications
You must be signed in to change notification settings - Fork 255
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
[BUG] Accessibility tags missing on download #552
Comments
Welcome! Thanks for posting your first issue. The way things work here is that while customer issues are prioritized, other issues go into our backlog where they are assessed and fitted into the roadmap when suitable. If you need to get this done, consider buying a license which also enables you to use it in your commercial products. More information can be found on https://unidoc.io/ |
We have a license |
Dear @vikraman-alea-bah Thank you for providing such a detailed report on the issue. We have investigated it thoroughly and will get back to you with a response as soon as possible. Thank you for your patience and understanding. |
We have investigated this matter further and can confirm that UniPDF does not yet fully support accessibility features. However, we are committed to addressing this issue and have added it to our roadmap for future development. Our goal is to enhance the accessibility support for PDF documents. Based on you attached document, we observed that some accessibility checks done in Adobe Acrobat are failing while others are passing. Nevertheless, we understand the importance of ensuring that every feature is properly implemented to generate documents with comprehensive accessibility support. |
Thanks for looking into it @3ace! The most important thing is the preservation of the accessibility tags in order. Without that, we can't say the PDF is accessible. Do you have a public facing roadmap that I can share with my team? PDF accessibility has become a requirement for us and it would be great if we could follow it on the roadmap to get an idea of when to expect it. |
@vikraman-alea-bah Continuing from our last update, we've tried to investigate what needs to be improved on to support generating accessibility documents. We've started by fixing the form flattening process to preserve all accessibility information. Currently, we are in the process of creating an internal roadmap and do not have a public-facing version available. However, I can assure you that PDF accessibility is a priority for us. |
Hi @vikraman-alea-bah we have made some updates that should helps to preserve an existing accessibility features when processing existing PDF document. Here is the accessibility reports generated by Adobe Acrobat before and after the implementation.
The fix should be available with our next UniPDF release version 3.61.0 that should be released next month. |
Thank you so much for this! I also checked it against adobe acrobat pro and all the accessibility tags were preserved and in order. Amazing work!! |
Hi @3ace I saw that the 3.61.0 got released, did these changes make it in? My team plans on integrating it soon. |
@vikraman-alea-bah yes, the update should have been included with the release. Let us know if you have further issue. |
@3ace my team has been testing with the latest release and we are still seeing the issue where the accessibility tags are removed on download. We have to flatten the files so they cannot be edited after download. Just wondering if you tested with anything like that and if it worked for you. |
hi @vikraman-alea-bah thanks for reaching out. Is the document that has the issue is the same document attached above? If it's happening in other files, it would be great if you could attach it here. We did test it using a flattening process using this code example https://github.com/unidoc/unipdf-examples/blob/master/forms/pdf_form_flatten.go |
@3ace thank you for giving us this example so quickly! We used your code example to help us narrow down where we think the issue is. We successfully got the accessibility tags to stay after flattening in most scenarios. However, there may be a bug in the Here is a code example of the two scenarios we tried that did not keep the accessibility tags:
|
@vikraman-alea-bah just to clarify the flow, are you adding the page after or before flattening the document? |
We add the pages after flattening. However, we tested after and before and in both cases it didn't include the accessibility tags. |
@vikraman-alea-bah thanks for the confirmation. We'll take a look at it |
@vikraman-alea-bah The issue you're experiencing is due to the current limitations in UniPDF. At the moment, UniPDF doesn't support adding new accessibility tags; it only support existing ones. This means that when a new page without accessibility tags is added to a PDF document that already has them, the final document will end up with incomplete accessibility tags. However, if the new page you're adding already includes accessibility tags, the resulting PDF should have a complete set of tags. We are actively working on adding support for new accessibility tags and will notify you once it's ready. |
Thank you for looking into that @3ace ! |
Good morning @3ace. Sorry for the back and forth on this one. Just wanted to make sure that we aren't doing something wrong on our end with that last post that @vikraman-alea-bah made. When using that script that you all were using for testing https://github.com/unidoc/unipdf-examples/blob/master/forms/pdf_form_flatten.go it seemed like using the Now when we changed this up to using the Creator or PdfWriter packages to write the files for whatever reason the accessibility tags were then removed. After reading in a page with pdfReader. (from a pdf that had the tags) Providing short example below.
Just wanted to confirm that this should in theory result in the pdf being output to contain the original accessibility tags. Thanks! |
@Preziotti-Matthew-bah @vikraman-alea-bah there's a difference between using
So if you would like to not use func process(inputPath, outputPath string) error {
f, err := os.Open(inputPath)
if err != nil {
return err
}
defer f.Close()
pdfReader, err := model.NewPdfReader(f)
if err != nil {
return err
}
page, err := pdfReader.GetPage(1)
if err != nil {
return err
}
pdfWriter := model.NewPdfWriter()
err = pdfWriter.AddPage(page)
if err != nil {
return err
}
// Copy PDF version.
version := pdfReader.PdfVersion()
pdfWriter.SetVersion(version.Major, version.Minor)
// Copy PDF info.
info, err := pdfReader.GetPdfInfo()
if err != nil {
common.Log.Debug("ERROR: %v", err)
} else {
pdfWriter.SetDocInfo(info)
}
// Copy Catalog Metadata.
if meta, ok := pdfReader.GetCatalogMetadata(); ok {
if err := pdfWriter.SetCatalogMetadata(meta); err != nil {
return err
}
}
// Copy catalog mark information.
if markInfo, ok := pdfReader.GetCatalogMarkInfo(); ok {
if err := pdfWriter.SetCatalogMarkInfo(markInfo); err != nil {
return err
}
}
// Copy AcroForm.
err = pdfWriter.SetForms(pdfReader.AcroForm)
if err != nil {
common.Log.Debug("ERROR: %v", err)
return err
}
// Copy viewer preferences.
if pref, ok := pdfReader.GetCatalogViewerPreferences(); ok {
if err := pdfWriter.SetCatalogViewerPreferences(pref); err != nil {
return err
}
}
// Copy language preferences.
if lang, ok := pdfReader.GetCatalogLanguage(); ok {
if err := pdfWriter.SetCatalogLanguage(lang); err != nil {
return err
}
}
// Copy document outlines.
pdfWriter.AddOutlineTree(pdfReader.GetOutlineTree())
// Copy OC Properties.
props, err := pdfReader.GetOCProperties()
if err != nil {
common.Log.Debug("ERROR: %v", err)
} else {
err = pdfWriter.SetOCProperties(props)
if err != nil {
common.Log.Debug("ERROR: %v", err)
}
}
// Copy page labels.
labelObj, err := pdfReader.GetPageLabels()
if err != nil {
common.Log.Debug("ERROR: %v", err)
} else {
err = pdfWriter.SetPageLabels(labelObj)
if err != nil {
common.Log.Debug("ERROR: %v", err)
}
}
// Copy named destinations.
namedDest, err := pdfReader.GetNamedDestinations()
if err != nil {
common.Log.Debug("ERROR: %v", err)
} else {
err = pdfWriter.SetNamedDestinations(namedDest)
if err != nil {
common.Log.Debug("ERROR: %v", err)
}
}
// Copy name dictionary.
nameDict, err := pdfReader.GetNameDictionary()
if err != nil {
common.Log.Debug("ERROR: %v", err)
} else {
err = pdfWriter.SetNameDictionary(nameDict)
if err != nil {
common.Log.Debug("ERROR: %v", err)
}
}
structTreeRoot, found := pdfReader.GetCatalogStructTreeRoot()
if found {
err := pdfWriter.SetCatalogStructTreeRoot(structTreeRoot)
if err != nil {
common.Log.Debug("ERROR: %v", err)
}
}
// Copy global page rotation.
if pdfReader.Rotate != nil {
if err := pdfWriter.SetRotation(*pdfReader.Rotate); err != nil {
common.Log.Debug("ERROR: %v", err)
}
}
err = pdfWriter.WriteToFile(outputPath)
if err != nil {
return err
}
return nil
} I hope this helps |
@3ace Thank you for this, this is extremely insightful! The main reason we are using the Creator is that we need to draw an image on the pdf (add a QR code). This code was written years ago and most of the examples I've seen use the Creator package. Would the best way forward be to continue using the Creator and then add the metadata the way you have done above? Or is there a better way to add an image using PdfWriter? Thanks again for all the support. |
@Preziotti-Matthew-bah You can keep using Creator package since most of our functionality is available using that package (for example to add image to PDF), the PdfWriter package contains some other functionality so you could use whichever necessary depending on your need. |
@3ace we have made some good progress on our end, but still have a couple issues tripping us up.
Thanks again. |
|
Hey @3ace, Sorry probably didn't do a good job of phrasing question 1. Here is an example situation for that:
Hopefully that explains it a bit better. Thanks as well for the link to the documentation. |
Hi @Preziotti-Matthew-bah before we are continuing, my colleague should have sent invitation mail to @vikraman-alea-bah so that you could post a request trough our service desk board where we could better monitor the request and you might be feel more comfortable with sharing some information there. |
thank you for all of your help @3ace it has really helped us in moving forward while we try to solve this issue. Can you also please send the service desk link to @Preziotti-Matthew-bah he is the developer working on this so will be able to speak more technically about the issue. |
@Preziotti-Matthew-bah could you provide me the email address for the invitation email? |
@3ace sure thing. That will go to [email protected]. Thanks! |
@3ace has this been sent over already? Just want to make sure I haven't missed it. |
@Preziotti-Matthew-bah sorry that just see this reply. I saw a ticket created by you, so I guess you've received the invitation right? |
Yeah we got all squared away 👍 Thanks! |
Description
Accessibility tags allow blind and low vision screen reader users the ability to access the information on PDFs. On download, the PDF needs to preserve the accessibility tags (and their order), the alternative text in images, and the title of the document. Without the tags a blind screen reader user cannot access the information in a human-readable way (if at all) on a digital PDF.
Bug:
A PDF with accessibility tags that is flattened and downloaded using unipdf loses some accessibility features such as accessibility tags and alternative text for images
.
Expected Behavior
Accessibility tags (in order) and alt text for images are preserved on download
How to test
You can test if a PDF has accessibility tags a few ways.
Attachments
Attached you'll find:
adobe acrobat pro
adobe acrobat free
pdf with accessibility tags
accessible-pdf.pdf
pdf downloaded using unipdf (with lost accessibility tags)
output.pdf
Code
The text was updated successfully, but these errors were encountered: