From 09df98b2b47f6f9a754d5ff8a1787da5a07c14b4 Mon Sep 17 00:00:00 2001 From: Carl Hetherington Date: Sun, 1 May 2022 19:35:26 +0200 Subject: [PATCH] Make sure that crop preview rectangle is properly hidden when we close the auto crop dialog. --- src/wx/content_menu.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/wx/content_menu.cc b/src/wx/content_menu.cc index ed515f6c8..93bcc0a6d 100644 --- a/src/wx/content_menu.cc +++ b/src/wx/content_menu.cc @@ -602,13 +602,13 @@ ContentMenu::auto_crop () /* Handle the user closing the dialog (with OK or cancel) */ _auto_crop_dialog->Bind (wxEVT_BUTTON, [this, viewer](wxCommandEvent& ev) { + _auto_crop_config_connection.disconnect (); + _auto_crop_viewer_connection.disconnect (); if (ev.GetId() == wxID_OK) { _content.front()->video->set_crop(_auto_crop_dialog->get()); } _auto_crop_dialog->Show (false); viewer->unset_crop_guess (); - _auto_crop_config_connection.disconnect (); - _auto_crop_viewer_connection.disconnect (); }); /* Update the view when something in the dialog is changed */ -- 2.30.2