From ff0e8de420650b4467fbea8d188aad8481415210 Mon Sep 17 00:00:00 2001
From: abose <arun@core.ai>
Date: Wed, 25 Aug 2021 23:24:45 +0530
Subject: [PATCH] Fix theme loading issues
 https://github.com/aicore/phoenix/issues/26

---
 src/extensions/default/DefaultExtensions.json |  4 +-
 src/view/ThemeManager.js                      | 62 +++++++++++++++++++
 2 files changed, 65 insertions(+), 1 deletion(-)

diff --git a/src/extensions/default/DefaultExtensions.json b/src/extensions/default/DefaultExtensions.json
index 73142f279f..41c5f62548 100644
--- a/src/extensions/default/DefaultExtensions.json
+++ b/src/extensions/default/DefaultExtensions.json
@@ -27,5 +27,7 @@
   "RecentProjects",
   "RemoteFileAdapter",
   "SVGCodeHints",
-  "UrlCodeHints"
+  "UrlCodeHints",
+  "DarkTheme",
+  "LightTheme"
 ]
diff --git a/src/view/ThemeManager.js b/src/view/ThemeManager.js
index c0e1fc2d1b..17907f0ff7 100644
--- a/src/view/ThemeManager.js
+++ b/src/view/ThemeManager.js
@@ -270,6 +270,64 @@ define(function (require, exports, module) {
         });
     }
 
+    /**
+     *
+     * @param file FileSystem.getFileForPath object
+     * @param options
+     * @private
+     */
+    function _loadThemeFromFile(file, options) {
+        var currentThemeName = prefs.get("theme");
+        var theme = new Theme(file, options);
+        loadedThemes[theme.name] = theme;
+        ThemeSettings._setThemes(loadedThemes);
+
+        // For themes that are loaded after ThemeManager has been loaded,
+        // we should check if it's the current theme.  If it is, then we just
+        // load it.
+        if (currentThemeName === theme.name) {
+            refresh(true);
+        }
+    }
+
+    /**
+     * Loads a theme from a url.
+     *
+     * @param {string} url is the full hhtp/https url of the theme file
+     * @param {Object} options is an optional parameter to specify metadata
+     *    for the theme.
+     * @return {$.Promise} promise object resolved with the theme to be loaded from fileName
+     */
+    function _loadFileFromURL(url, options) {
+        var deferred         = new $.Deferred();
+
+        var themeName = options.name || options.theme.title;
+        var fileName = options.theme.file;
+        var themePath = path.normalize(brackets.app.getApplicationSupportDirectory() + "/extensions/user/" +
+            themeName + '_' + fileName);
+        var file = FileSystem.getFileForPath(themePath);
+
+        file.exists(function (err, exists) {
+            var theme;
+
+            if (exists) {
+                _loadThemeFromFile(file, options);
+                deferred.resolve(theme);
+                return;
+            }
+            $.get(url).done(function (themeContent) {
+                // Write theme to file
+                FileUtils.writeText(file, themeContent).then(function () {
+                    _loadThemeFromFile(file, options);
+                    deferred.resolve(theme);
+                }, deferred.reject);
+            }).fail(function (err) {
+                deferred.reject(err);
+            });
+        });
+
+        return deferred.promise();
+    }
 
     /**
      * Loads a theme from a file.
@@ -280,6 +338,10 @@ define(function (require, exports, module) {
      * @return {$.Promise} promise object resolved with the theme to be loaded from fileName
      */
     function loadFile(fileName, options) {
+        if(fileName.startsWith("http://") || fileName.startsWith("https://")) {
+            return _loadFileFromURL(fileName, options);
+        }
+
         var deferred         = new $.Deferred(),
             file             = FileSystem.getFileForPath(fileName),
             currentThemeName = prefs.get("theme");