Fix/build output layout (#3716)

* fix: `ol.css` for nwjs build

* fix: global vars broken by race condition

* fix: only copy node modules css

Author: chmelevskij

gulpfile.js

@@ -380,14 +380,14 @@ function dist_src() {
 
 function dist_node_modules_css() {
   return gulp
-    .src("./**/*.min.css")
-    .pipe(gulp.dest(DIST_DIR));
+    .src("./node_modules/**/*.min.css")
+    .pipe(gulp.dest(`${DIST_DIR}node_modules`));
 }
 
 function dist_ol_css() {
     return gulp
         .src("./node_modules/ol/ol.css", { base: "node_modules" })
-        .pipe(gulp.dest(DIST_DIR));
+        .pipe(gulp.dest(`${DIST_DIR}css/tabs/`));
 }
 
 function dist_less() {

src/index.html

@@ -337,6 +337,5 @@ <h3 class="dialogInformationTitle"></h3>
             <a href="#" class="dialogInformation-confirmButton regular-button"></a>
         </div>
     </dialog>
-    <!-- CORDOVA_INCLUDE cordova.js -->
 </body>
 </html>

src/js/main.js

@@ -56,9 +56,12 @@ function readConfiguratorVersionMetadata() {
         CONFIGURATOR.version = manifest.version;
         CONFIGURATOR.gitRevision = manifest.gitRevision;
     } else {
-        CONFIGURATOR.productName = __APP_PRODUCTNAME__;
-        CONFIGURATOR.version = __APP_VERSION__;
-        CONFIGURATOR.gitRevision = __APP_REVISION__;
+        // These are injected by vite. If not checking
+        // for undefined occasionally there is a race
+        // condition where this fails the nwjs and cordova builds
+        CONFIGURATOR.productName = typeof __APP_PRODUCTNAME__ !== 'undefined' ? __APP_PRODUCTNAME__ : 'Betaflight Configurator';
+        CONFIGURATOR.version = typeof __APP_VERSION__ !== 'undefined' ? __APP_VERSION__ : '0.0.0';
+        CONFIGURATOR.gitRevision = typeof __APP_REVISION__ !== 'undefined' ? __APP_REVISION__ : 'unknown';
     }
 }