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

Fixed handling of transparency in PDF/A mode in addExtGState method #692

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

Conversation

stollr
Copy link

@stollr stollr commented Mar 11, 2024

The condition allowed to add ExtGState in all PDF/A modes and disallowed it in default mode until now. This looks logically wrong.

This fix inlines the condition with setExtGState to allow transparency parameters for non-PDF/A and PDF/A > 1 documents.

The state condition is copied from 'setExtGState'.

@CLAassistant
Copy link

CLAassistant commented Mar 11, 2024

CLA assistant check
All committers have signed the CLA.

@williamdes
Copy link
Contributor

@nicolaasuni what do you think ?

@nicolaasuni
Copy link
Member

Please try to resolve the conflicts.

@nicolaasuni nicolaasuni reopened this Apr 20, 2024
The condition allowed to add ExtGState in all PDF/A modes and disallowed
in default mode.

This fix inlines the condition with setExtGState to allow transparency
parameters for non-PDF/A and PDF/A > 1 documents.

The state condition is copied from 'setExtGState'.
@stollr
Copy link
Author

stollr commented Apr 28, 2024

@nicolaasuni I have rebased the PR on the main branch

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.

None yet

4 participants