fix: Remove platform theme url from stylesheets, (#761)

* Remove platform theme url from stylesheets, to prevent user agent from being overwritten

* fix typo

* Fix preview

Author: jperedadnr

kit/src/main/java/com/oracle/javafx/scenebuilder/kit/editor/EditorPlatform.java

@@ -159,6 +159,11 @@ public static Theme valueOf(String themeName) {
         // no-op
     }
 
+    public static boolean isPlatformThemeStylesheetURL(String stylesheetURL) {
+        // Return USER_AGENT css, which is Modena for fx 8.0
+        return stylesheetURL != null && stylesheetURL.equals(Theme.MODENA.getStylesheetURLs().getFirst());
+    }
+
     public static String getPlatformThemeStylesheetURL() {
         // Return USER_AGENT css, which is Modena for fx 8.0
         return Theme.MODENA.getStylesheetURLs().getFirst();

kit/src/main/java/com/oracle/javafx/scenebuilder/kit/editor/panel/content/ContentPanelController.java

@@ -1075,7 +1075,7 @@ private void themeDidChange() {
             final EditorPlatform.Theme theme = getEditorController().getTheme();
             List<String> themeStylesheets = new ArrayList<>(EditorPlatform.getStylesheetsForTheme(theme));
             themeStylesheets.addAll(theme.getStylesheetURLs());
-            workspaceController.setThemeStyleSheet(themeStylesheets, theme);
+            workspaceController.setThemeStylesheet(themeStylesheets, theme);
         }
     }
 

kit/src/main/java/com/oracle/javafx/scenebuilder/kit/editor/panel/content/WorkspaceController.java

@@ -163,16 +163,18 @@ public List<String> getThemeStyleSheets() {
         return Collections.unmodifiableList(themeStylesheets);
     }
 
-    public void setThemeStyleSheet(List<String> themeStyleSheets, EditorPlatform.Theme theme) {
-        assert themeStyleSheets != null;
+    public void setThemeStylesheet(List<String> themeStylesheets, EditorPlatform.Theme theme) {
+        assert themeStylesheets != null;
         assert theme != null;
         List<String> stylesheets = new ArrayList<>(EditorPlatform.getStylesheetsForTheme(theme));
-        stylesheets.addAll(themeStyleSheets);
-        contentSubScene.setUserAgentStylesheet(stylesheets.getFirst());
+        themeStylesheets.stream()
+            .filter(s -> !EditorPlatform.isPlatformThemeStylesheetURL(s))
+            .forEach(stylesheets::add);
+        contentSubScene.setUserAgentStylesheet(stylesheets.stream().findFirst().orElse(null));
 
-        ObservableList<String> currentStyleSheets = FXCollections.observableArrayList(stylesheets);
-        themeStylesheets.clear();
-        themeStylesheets.addAll(currentStyleSheets);
+        ObservableList<String> currentStylesheets = FXCollections.observableArrayList(stylesheets);
+        this.themeStylesheets.clear();
+        this.themeStylesheets.addAll(currentStylesheets);
         contentGroupApplyCss();
 
         // Update scenegraph layout, etc

kit/src/main/java/com/oracle/javafx/scenebuilder/kit/preview/PreviewWindowController.java

@@ -271,7 +271,9 @@ public void run() {
 
                     Object sceneGraphRoot = clone.getDisplayNodeOrSceneGraphRoot();
                     themeStyleSheetsList = new ArrayList<>(EditorPlatform.getStylesheetsForTheme(editorController.getTheme()));
-                    themeStyleSheetsList.addAll(editorControllerTheme.getStylesheetURLs());
+                    editorControllerTheme.getStylesheetURLs().stream()
+                        .filter(s -> !EditorPlatform.isPlatformThemeStylesheetURL(s))
+                            .forEach(themeStyleSheetsList::add);
 
                     if (sceneGraphRoot instanceof Parent) {
                         ((Parent) sceneGraphRoot).setId(NID_PREVIEW_ROOT);

kit/src/main/java/com/oracle/javafx/scenebuilder/kit/util/CssInternal.java

@@ -46,6 +46,7 @@
 import java.util.TreeMap;
 import java.util.TreeSet;
 
+import com.oracle.javafx.scenebuilder.kit.editor.EditorPlatform;
 import javafx.beans.property.ReadOnlyProperty;
 import javafx.collections.FXCollections;
 import javafx.css.CssMetaData;
@@ -151,10 +152,12 @@ public static boolean isThemeClass(Theme theme, String styleClass) {
 
     public static List<String> getThemeStyleClasses(Theme theme) {
         Set<String> themeClasses = new HashSet<>();
-        theme.getStylesheetURLs().forEach(themeStyleSheet -> {
-            URL resource = Button.class.getResource("/" + themeStyleSheet);
-            themeClasses.addAll(getStyleClasses(resource));
-        });
+        theme.getStylesheetURLs().stream()
+            .filter(s -> !EditorPlatform.isPlatformThemeStylesheetURL(s))
+            .forEach(themeStyleSheet -> {
+                URL resource = Button.class.getResource("/" + themeStyleSheet);
+                themeClasses.addAll(getStyleClasses(resource));
+            });
         return new ArrayList<>(themeClasses);
     }