Added an icon for the Non-Owned Script Settings Menu.

(also I made closing that menu actually save the script, my bad)

Author: XTerPL

src/main/java/io/github/techstreet/dfscript/screen/script/ScriptListScreen.java

@@ -214,7 +214,7 @@ public boolean mouseClicked(double x, double y, int button) {
                 }
                 else {
                     //Script Settings Button
-                    CButton settings = new CTexturedButton(10 + addedX, y + addedY, 8, 8, DFScript.MOD_ID + ":wrench.png", () -> {
+                    CButton settings = new CTexturedButton(10 + addedX, y + addedY, 8, 8, DFScript.MOD_ID + ":settings.png", () -> {
                         DFScript.MC.setScreen(new ScriptSettingsScreen(s, false));
                     }, 0, 0, 1, 0.5f, 0, 0.5f);
 

src/main/java/io/github/techstreet/dfscript/screen/script/ScriptSettingsScreen.java

@@ -137,7 +137,13 @@ public boolean mouseClicked(double x, double y, int button) {
 
     @Override
     public void close() {
-        DFScript.MC.setScreen(owned ? new ScriptEditScreen(script) : new ScriptListScreen(true));
+        if (owned) {
+            DFScript.MC.setScreen(new ScriptEditScreen(script));
+        }
+        else {
+            ScriptManager.getInstance().saveScript(script);
+            DFScript.MC.setScreen(new ScriptListScreen(true));
+        }
     }
 
     public boolean mouseClicked(double mouseX, double mouseY, int button) {

src/main/java/io/github/techstreet/dfscript/script/options/ScriptNamedOption.java

@@ -6,6 +6,7 @@
 import io.github.techstreet.dfscript.script.Script;
 import io.github.techstreet.dfscript.script.ScriptPart;
 import io.github.techstreet.dfscript.script.argument.ScriptArgument;
+import io.github.techstreet.dfscript.util.chat.ChatUtil;
 import net.minecraft.item.ItemStack;
 import net.minecraft.text.Style;
 import net.minecraft.text.Text;
@@ -77,7 +78,9 @@ public JsonElement serialize(ScriptNamedOption src, Type typeOfSrc, JsonSerializ
 
             object.addProperty("type", src.getOption().getType());
 
-            object.add("value", src.getOption().getJsonPrimitive());
+            JsonPrimitive primitive = src.getOption().getJsonPrimitive();
+
+            object.add("value", primitive);
 
             return object;
         }

src/main/java/io/github/techstreet/dfscript/script/options/ScriptOption.java

@@ -1,6 +1,7 @@
 package io.github.techstreet.dfscript.script.options;
 
 import com.google.gson.JsonElement;
+import com.google.gson.JsonPrimitive;
 import io.github.techstreet.dfscript.screen.widget.CScrollPanel;
 import io.github.techstreet.dfscript.script.argument.ScriptArgument;
 import net.minecraft.item.Item;
@@ -16,5 +17,5 @@ public interface ScriptOption {
 
     String getType();
 
-    JsonElement getJsonPrimitive();
+    JsonPrimitive getJsonPrimitive();
 }

src/main/java/io/github/techstreet/dfscript/script/options/ScriptTextOption.java

@@ -48,7 +48,7 @@ public String getType() {
     }
 
     @Override
-    public JsonElement getJsonPrimitive() {
+    public JsonPrimitive getJsonPrimitive() {
         return new JsonPrimitive(value);
     }
 }