Size limit check must include document formats

Issue #519 resolved
Robert Leach created an issue

USE CASE: WHAT DO YOU WANT TO DO?

Be informed in a graceful way when an image export will not work due to size.

STEPS TO REPRODUCE AN ISSUE (OR TRIGGER A NEW FEATURE)

  1. Open a very large matrix that will result in a matrix whose dimension multiplication times 9 will be larger than Integer.MAX_VALUE
  2. Export a PDF

CURRENT BEHAVIOR

Because the resolution of issue #409 (large PDF file size) was to embed a PNG in a PDF, and because that PNG is subject to the same size limitations as a regular image format, the size checks imposed on PNGs in the export dialog must now also be applied to document formats (such as PDFs). Currently, if a document format is selected in the export interface, all options are all always enabled by default without size checks. If options are selected result in a matrix that is too big, there will be an exception. See issue #383 for an explanation of the size limitation.

EXPECTED BEHAVIOR

Size checks are performed in document format on the matrix only (since that's the only PNG component) and options in the export interface which would result in a matrix image that is too large will be disabled. Whenever a selection in the export interface changes that affects to dimensions of the PNG component, the checks should be redone, and the enabled state of the interface elements refreshed.

DEVELOPERS ONLY SECTION

SUGGESTED CHANGE (Pseudocode optional)

Add checks on the matrix size wherever size checks are currently skipped when document format is selected.

FILES AFFECTED (where the changes will be implemented) - developers only

unknown

LEVEL OF EFFORT - developers only

medium

COMMENTS

Comments (1)

  1. Log in to comment