From b2f77fd554588ae2bb8c5281959130c734aa33e4 Mon Sep 17 00:00:00 2001 From: Richard Cordovano Date: Tue, 8 Sep 2020 17:52:32 -0400 Subject: [PATCH] 6807 Tidy up MediaViewImagePanel --- .../autopsy/contentviewers/MediaViewImagePanel.java | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/Core/src/org/sleuthkit/autopsy/contentviewers/MediaViewImagePanel.java b/Core/src/org/sleuthkit/autopsy/contentviewers/MediaViewImagePanel.java index d45a7c057e..ba5f1ef8e9 100644 --- a/Core/src/org/sleuthkit/autopsy/contentviewers/MediaViewImagePanel.java +++ b/Core/src/org/sleuthkit/autopsy/contentviewers/MediaViewImagePanel.java @@ -358,11 +358,11 @@ class MediaViewImagePanel extends JPanel implements MediaFileViewer.MediaViewPan private void showErrorNode(String errorMessage, AbstractFile file) { final Button externalViewerButton = new Button(Bundle.MediaViewImagePanel_externalViewerButton_text(), new ImageView(externalImage)); /* - * TODO: why is the name passed into the action constructor? it means we - * duplicate this string all over the place -jm + * Tie a Swing action (ExternalViewerAction) to a JFX button action. */ - externalViewerButton.setOnAction(actionEvent -> new ExternalViewerAction(Bundle.MediaViewImagePanel_externalViewerButton_text(), new FileNode(file)) - .actionPerformed(new ActionEvent(this, ActionEvent.ACTION_PERFORMED, "")) //Swing ActionEvent + externalViewerButton.setOnAction(actionEvent -> + new ExternalViewerAction(Bundle.MediaViewImagePanel_externalViewerButton_text(), new FileNode(file)) + .actionPerformed(new ActionEvent(this, ActionEvent.ACTION_PERFORMED, "")) ); final VBox errorNode = new VBox(10, new Label(errorMessage), externalViewerButton);