Merge pull request #2742 from haslinghuis/remove_backup_restore

Author: blckmn

src/js/tabs/setup.js

@@ -29,18 +29,34 @@ TABS.setup.initialize = function (callback) {
         // translate to user-selected language
         i18n.localizePage();
 
-        const backupButton = $('#content .backup');
+        if (CONFIGURATOR.virtualMode || semver.lt(FC.CONFIG.apiVersion, API_VERSION_1_44)) {
+            const backupButton = $('#content .backup');
 
-        if (semver.lt(FC.CONFIG.apiVersion, CONFIGURATOR.API_VERSION_MIN_SUPPORTED_BACKUP_RESTORE)) {
-            backupButton.addClass('disabled');
-            $('#content .restore').addClass('disabled');
+            if (semver.lt(FC.CONFIG.apiVersion, CONFIGURATOR.API_VERSION_MIN_SUPPORTED_BACKUP_RESTORE)) {
+                backupButton.addClass('disabled');
+                $('#content .restore').addClass('disabled');
 
-            GUI.log(i18n.getMessage('initialSetupBackupAndRestoreApiVersion', [FC.CONFIG.apiVersion, CONFIGURATOR.API_VERSION_MIN_SUPPORTED_BACKUP_RESTORE]));
-        }
+                GUI.log(i18n.getMessage('initialSetupBackupAndRestoreApiVersion', [FC.CONFIG.apiVersion, CONFIGURATOR.API_VERSION_MIN_SUPPORTED_BACKUP_RESTORE]));
+            }
+
+            if (CONFIGURATOR.virtualMode) {
+                // saving and uploading an imaginary config to hardware is a bad idea
+                backupButton.addClass('disabled');
+            }
+
+            backupButton.on('click', () => configuration_backup(() => GUI.log(i18n.getMessage('initialSetupBackupSuccess'))));
 
-        // saving and uploading an imaginary config to hardware is a bad idea
-        if (CONFIGURATOR.virtualMode) {
-            backupButton.addClass('disabled');
+            $('#content .restore').on('click', () => {
+                configuration_restore(() => {
+                    // get latest settings
+                    TABS.setup.initialize();
+
+                    GUI.log(i18n.getMessage('initialSetupRestoreSuccess'));
+                });
+            });
+
+        } else {
+            $('.backupRestore').hide();
         }
 
         // initialize 3D Model
@@ -164,29 +180,6 @@ TABS.setup.initialize = function (callback) {
             console.log(`YAW reset to 0 deg, fix: ${self.yaw_fix} deg`);
         });
 
-        backupButton.click(function () {
-            if ($(this).hasClass('disabled')) {
-                return;
-            }
-
-            configuration_backup(function () {
-                GUI.log(i18n.getMessage('initialSetupBackupSuccess'));
-            });
-        });
-
-        $('#content .restore').click(function () {
-            if ($(this).hasClass('disabled')) {
-                return;
-            }
-
-            configuration_restore(function () {
-                // get latest settings
-                TABS.setup.initialize();
-
-                GUI.log(i18n.getMessage('initialSetupRestoreSuccess'));
-            });
-        });
-
         // cached elements
         const bat_voltage_e = $('.bat-voltage'),
             bat_mah_drawn_e = $('.bat-mah-drawn'),

src/tabs/setup.html

@@ -55,7 +55,7 @@
                 </div>
             </div>
         </div>
-        <div class="grid-row">
+        <div class="grid-row backupRestore">
             <div class="grid-col col3">
                 <div class="grid-row">
                     <div class="grid-col col6">