From 5f2f439fc3709c8448d61225f287a3817be11eb9 Mon Sep 17 00:00:00 2001
From: Leon Styhre <leon@leonstyhre.com>
Date: Sun, 16 Jan 2022 10:43:12 +0100
Subject: [PATCH] Moved the fan art scrape menu option to the bottom and set it
 as disabled by default.

---
 es-app/src/guis/GuiScraperMenu.cpp | 22 +++++++++++-----------
 es-core/src/Settings.cpp           |  2 +-
 2 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/es-app/src/guis/GuiScraperMenu.cpp b/es-app/src/guis/GuiScraperMenu.cpp
index 636f5fd98..1b234c316 100644
--- a/es-app/src/guis/GuiScraperMenu.cpp
+++ b/es-app/src/guis/GuiScraperMenu.cpp
@@ -353,17 +353,6 @@ void GuiScraperMenu::openContentOptions()
         }
     });
 
-    // Scrape fan art images.
-    auto scrapeFanArt = std::make_shared<SwitchComponent>(mWindow);
-    scrapeFanArt->setState(Settings::getInstance()->getBool("ScrapeFanArt"));
-    s->addWithLabel("FAN ART IMAGES", scrapeFanArt);
-    s->addSaveFunc([scrapeFanArt, s] {
-        if (scrapeFanArt->getState() != Settings::getInstance()->getBool("ScrapeFanArt")) {
-            Settings::getInstance()->setBool("ScrapeFanArt", scrapeFanArt->getState());
-            s->setNeedsSaving();
-        }
-    });
-
     // Scrape marquee images.
     auto scrape_marquees = std::make_shared<SwitchComponent>(mWindow);
     scrape_marquees->setState(Settings::getInstance()->getBool("ScrapeMarquees"));
@@ -419,6 +408,17 @@ void GuiScraperMenu::openContentOptions()
             ->setOpacity(DISABLED_OPACITY);
     }
 
+    // Scrape fan art images.
+    auto scrapeFanArt = std::make_shared<SwitchComponent>(mWindow);
+    scrapeFanArt->setState(Settings::getInstance()->getBool("ScrapeFanArt"));
+    s->addWithLabel("FAN ART IMAGES", scrapeFanArt);
+    s->addSaveFunc([scrapeFanArt, s] {
+        if (scrapeFanArt->getState() != Settings::getInstance()->getBool("ScrapeFanArt")) {
+            Settings::getInstance()->setBool("ScrapeFanArt", scrapeFanArt->getState());
+            s->setNeedsSaving();
+        }
+    });
+
     mWindow->pushGui(s);
 }
 
diff --git a/es-core/src/Settings.cpp b/es-core/src/Settings.cpp
index 3eb6276fa..115408074 100644
--- a/es-core/src/Settings.cpp
+++ b/es-core/src/Settings.cpp
@@ -97,10 +97,10 @@ void Settings::setDefaults()
     mBoolMap["ScrapeTitleScreens"] = {true, true};
     mBoolMap["ScrapeCovers"] = {true, true};
     mBoolMap["ScrapeBackCovers"] = {true, true};
-    mBoolMap["ScrapeFanArt"] = {true, true};
     mBoolMap["ScrapeMarquees"] = {true, true};
     mBoolMap["Scrape3DBoxes"] = {true, true};
     mBoolMap["ScrapePhysicalMedia"] = {true, true};
+    mBoolMap["ScrapeFanArt"] = {false, false};
 
     mStringMap["MiximageResolution"] = {"1280x960", "1280x960"};
     mStringMap["MiximageScreenshotScaling"] = {"sharp", "sharp"};