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.

Sign up for GitHub

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

Jump to bottom

Image to big #52

Closed
maticahacic opened this issue Sep 24, 2022 · 3 comments
Closed

Image to big #52

maticahacic opened this issue Sep 24, 2022 · 3 comments
Assignees
@cleydyr
Labels
bug

Comments

@maticahacic
Copy link

When I try to open a pdf I get Image to Big error.

Screenshot from 2022-09-24 11-05-57

Might be related to this issue - danmbox/briss2#5
@cleydyr
Copy link
Collaborator

cleydyr commented Sep 24, 2022

Hey, @maticahacic . Thank you for opening this issue.

Can you share an example PDF with which we can reproduce the behavior?

@cleydyr cleydyr self-assigned this Sep 24, 2022
@cleydyr cleydyr added the triage label Sep 24, 2022
@mbaeuerle
Copy link
Owner

Hey @maticahacic thanks for bringing this up, and also thanks for pointing to the related issue!
Seems like this is directly related to the PDF size. I could reproduce that with this Din A0 PDF (just white no content in case you're wondering). Let me check what we can do here!

@maticahacic
Copy link
Author

Great to hear about this!

mbaeuerle added a commit that referenced this issue Sep 24, 2022
@mbaeuerle
Fix #52 Remove PDF size restriction to allow crops of bigger files
49db2f6 
This could lead to unexpected results when using very big files. Din A0 seems to be handled fine though.
@cleydyr cleydyr added bug and removed triage labels Sep 24, 2022
cleydyr added a commit to cleydyr/Briss-2.0 that referenced this issue Nov 30, 2022
@cleydyr
Fix mbaeuerle#52 Remove PDF size restriction to allow crops of bigger…
83baf4a 
… files

This could lead to unexpected results when using very big files. Din A0 seems to be handled fine though.
cleydyr added a commit to cleydyr/Briss-2.0 that referenced this issue Nov 30, 2022
@cleydyr
Fix mbaeuerle#52 Remove PDF size restriction to allow crops of bigger…
6964c70 
… files

This could lead to unexpected results when using very big files. Din A0 seems to be handled fine though.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cleydyr cleydyr

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@cleydyr @mbaeuerle @maticahacic