Checking and unchecking "Show selections" in Export changes preview

[Anastasia Baryshnikova]

USE CASE: WHAT DO YOU WANT TO DO?

Make sure that the preview window in Export is consistent

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

Open any matrix, select a region, go File > Export, choose "All" and click/unclick "Show Selections".

CURRENT BEHAVIOR

The preview window changes slightly (some tiles appear, some disappear). I'm pretty sure this is inevitable because of the rendering of the image with and without the yellow selection box. But I figured I'll point it out in the remote case this is fixable.

EXPECTED BEHAVIOR

The preview window should be consistent with and without the selection box.

DEVELOPERS ONLY SECTION

SUGGESTED CHANGE (Pseudocode optional)

e.g. Add a color selection class

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

e.g. selectColor.java & settingsPanel.java

LEVEL OF EFFORT - developers only

medium

COMMENTS

[Srikanth Bezawada]

Hi @hepcat72,

The idea is to synchronize the preview image with the corresponding buttons selected. Whenever there is a change in selection of the buttons (region radiobuttons and "show selections" checkbox), the corresponding listener should call ExportPreviewMatrix.java to repaint accordingly. I have started working on this.

[Christopher Keil]

@srikanthbezawada Check pull request #64. There's a pretty large export preview update about to be merged.

[Srikanth Bezawada]

Hey @TreeView3Dev , Thanks for the info. Nevertheless, I got this working so that preview window changes according to the radiobutton selected All or Visible.

Looks like you've already covered this issue in that pull request.

[Robert Leach]

Duplicate of #370.

[Robert Leach]

This must be a test of a jar file that was generated from a branch the Chris is working on? I cannot reproduce this behavior. Checking the "Show selections" checkbox does not update the preview in any way in the master branch because the master has not had those preview updates implemented yet.

[Srikanth Bezawada]

Yes, @hepcat72 , I tested this on Chris's branch.

[Robert Leach]

Removing milestone: Import/export data (automated comment)