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

Exporting a large diagram: high zoom values not respected when exported image would have large dimensions #4304

Open
2 tasks done
axefrog opened this issue Apr 1, 2024 · 1 comment
Labels
blocked/need-repo Requires a complete test case that reproduces the problem

Comments

@axefrog
Copy link

axefrog commented Apr 1, 2024

Preflight Checklist

  • I agree to follow the Code of Conduct that this project adheres to.
  • I have searched the issue tracker for a feature request that matches the one I want to file, without success.

I'm using the latest version of the Windows 10 desktop version of the app. Here's a zoomed out screenshot of the diagram I'm trying to export:

image

This is what it looks like at 300% zoom in the app:

image

Here I am about to export the diagram as a PNG file:

image

And this is what the same section in the 300% zoomed image above looks like at 1:1 scaling in the exported PNG. I've annotated the screenshot with comparison to what was expected:

image

Eyeballing things, it looks like the actual exported zoom value used is about 170%, which incidentally is a zoom setting I frequently default to while editing. Not sure if there is some correlation there that I've missed, but I suspect it's a coincidence.

I have also tested at zoom values lower than 170% and the exported scale of the image seems about right, so it seems like something in the app is capping the export zoom value for some reason.

UPDATE:

I tried the "Advanced" export dialog and noticed that the dimension inputs are highlighted in red if the width of the exported image will be >= 24056. This limitation seems to be set by the app, as it is not a limitation of PNG or JPG files. Why is the limitation there? Does it need to be? I can see that maybe it might be required on the server side for the browser app, but the desktop app shouldn't have this limitation. If my intuition about the limitation being to save server-side resources in the diagrams.net app, can the limitation be disabled for the desktop release? If so, I'm happy to open an issue for the desktop repo to request this.

UPDATE:

The red highlighting of the input box doesn't seem to correlate with the refusal to export larger image sizes, as I've now tried exporting at lower (but still high) zoom values (e.g. 240%, which puts the exported width in the range of 22000-24000 pixels), and the exported image still seems to have a zoom value of about 170%.

draw.io version:

  • draw.io version 24.1.0

Desktop:

  • OS: Windows 10 Professional
  • Browser: Chrome
  • Browser Version: 123.0.6312.86 (Official Build) (64-bit)
  • ^^ Using the desktop version, not the browser version

I tested the problem in incognito/private mode with all browser extensions switched off, write "yes" below:

  • Not applicable (we're talking about is the desktop release)
@axefrog axefrog changed the title Exporting a large diagram: high zoom % value (300%-400%) not respected Exporting a large diagram: high zoom values not respected when exported image width will be >= 24000 pixels Apr 1, 2024
@axefrog axefrog changed the title Exporting a large diagram: high zoom values not respected when exported image width will be >= 24000 pixels Exporting a large diagram: high zoom values not respected when exported image has large dimensions Apr 1, 2024
@axefrog axefrog changed the title Exporting a large diagram: high zoom values not respected when exported image has large dimensions Exporting a large diagram: high zoom values not respected when exported image would have large dimensions Apr 1, 2024
@alderg
Copy link
Contributor

alderg commented Apr 6, 2024

Could you attach the diagram file and the exported PNG image please?

@davidjgraph davidjgraph added the blocked/need-repo Requires a complete test case that reproduces the problem label Apr 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked/need-repo Requires a complete test case that reproduces the problem
Projects
None yet
Development

No branches or pull requests

3 participants