From 2a4e7d22b8ce8d07485cfd6e28e482b065aeb730 Mon Sep 17 00:00:00 2001 From: Leon Styhre Date: Mon, 3 Jan 2022 18:44:19 +0100 Subject: [PATCH] Fixed an issue where the busy indicator would not get displayed during multi-scraping. --- es-app/src/guis/GuiScraperSearch.cpp | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/es-app/src/guis/GuiScraperSearch.cpp b/es-app/src/guis/GuiScraperSearch.cpp index 4e158c426..05eeef9e3 100644 --- a/es-app/src/guis/GuiScraperSearch.cpp +++ b/es-app/src/guis/GuiScraperSearch.cpp @@ -445,6 +445,15 @@ void GuiScraperSearch::onSearchDone(std::vector& results) mBlockAccept = false; updateInfoPane(); + // If there is a single result in semi-automatic mode or a single or more results in + // fully automatic mode, then block the ability to manually accept the entry as it will + // be selected as soon as the thumbnail has finished downloading. This also makes sure + // the busy animation will play during this time window. + if (!mRefinedSearch && + ((mSearchType == ACCEPT_SINGLE_MATCHES && results.size() == 1) || + (mSearchType == ALWAYS_ACCEPT_FIRST_RESULT && mScraperResults.size() > 0))) + mBlockAccept = true; + // If there is no thumbnail to download and we're in semi-automatic mode, proceed to return // the results or we'll get stuck forever waiting for a thumbnail to be downloaded. if (mSearchType == ACCEPT_SINGLE_MATCHES && results.size() == 1 &&