Fixed lag with window title colours being set too often

Author: rainlizard

Autoload/Constants.gd

@@ -2,9 +2,11 @@ extends Node
 const TILE_SIZE = 96
 const SUBTILE_SIZE = 32
 const PLAYERS_COUNT = 9
-#                      red                   blue                     green                  yellow                 white                    neutral              Purple                   Black                 Orange
-const ownerFloorCol = [Color8(132,44,0,255), Color8(136,112,148,255), Color8(52, 92, 4,255), Color8(188,156,0,255), Color8(207,207,207,255), Color8(52,36,4,255), Color8(171, 80,120,255), Color8(32,32,32,255), Color8(188,108,53,255)] #Color8(180,160,124,255)
-const ownerRoomCol =  [Color8(156,48,0,255), Color8(160,136,180,255), Color8(56,112,12,255), Color8(228,212,0,255), Color8(242,242,242,255), Color8(0,0,0,0),     Color8(200,104,164,255), Color8(1,1,1,255), Color8(211,132,72,255)] #Color8(188,168,132,255)
+#                      red                 blue                 green               yellow               white                neutral            Purple               Black             Orange
+const ownerFloorCol =  [Color8(132,44,0),  Color8(136,112,148), Color8(52,92,4),    Color8(188,156,0),   Color8(207,207,207), Color8(52,36,4),   Color8(171,80,120),  Color8(32,32,32), Color8(188,108,53)] #Color8(180,160,124,255)
+const ownerRoomCol =   [Color8(156,48,0),  Color8(160,136,180), Color8(56,112,12),  Color8(228,212,0),   Color8(242,242,242), Color8(0,0,0,0),   Color8(200,104,164), Color8(1,1,1),    Color8(211,132,72)] #Color8(188,168,132,255)
+const windowTitleCol = [Color8(153,92,92), Color8(151,118,168), Color8(107,128,84), Color8(178,173,116), Color8(184,184,184), Color8(86,82,102), Color8(153,92,121),  Color8(5,5,5),    Color8(178,134,89)]
+
 const ownershipNames = ["Red","Blue","Green","Yellow","White","None","Purple","Black","Orange"]
 
 const listOrientations = [

Scenes/OwnerSelection.gd

@@ -9,6 +9,7 @@ onready var oCurrentFormat = Nodelist.list["oCurrentFormat"]
 onready var gridItemScene = preload("res://Scenes/GenericGridItem.tscn")
 onready var oSelectedRect = $Control/SelectedRect
 onready var oCenteredLabel = $Control/CenteredLabel
+onready var oSelector = Nodelist.list["oSelector"]
 
 var ownership_available = true
 
@@ -117,7 +118,6 @@ func set_selection(value):
 	for id in oOwnershipGridContainer.get_children():
 		if id.get_meta("grid_value") == value:
 			oSelectedRect.boundToItem = id
-onready var oSelector = Nodelist.list["oSelector"]
 
 func update_ownership_options():
 	match oSelector.mode:

Scenes/Selection.gd

@@ -30,6 +30,7 @@ onready var oLoadingBar = Nodelist.list["oLoadingBar"]
 onready var oBrushPreview = Nodelist.list["oBrushPreview"]
 onready var oPlaceThingsAnywhere = Nodelist.list["oPlaceThingsAnywhere"]
 onready var oSlabSideViewer = Nodelist.list["oSlabSideViewer"]
+onready var oUseSlabOwnerCheckBox = Nodelist.list["oUseSlabOwnerCheckBox"]
 
 enum {
 	CONSTRUCT_BRUSH
@@ -86,9 +87,9 @@ func newPaintSubtype(value):
 
 
 func newOwnership(value):
-	oUi.update_theme_colour(value)
 	oOwnerSelection.set_selection(value)
 	paintOwnership = value
+	oUi.update_theme_colour(value)
 
 func newPaintSlab(value):
 	oSelector.change_mode(oSelector.MODE_TILE)
@@ -256,6 +257,9 @@ func place_subtile(placeSubtile):
 			return
 	oEditor.mapHasBeenEdited = true
 
+	if oUseSlabOwnerCheckBox.pressed == true and visible == true:
+		newOwnership(oDataOwnership.get_cellv_ownership(oSelector.cursorTile))
+	
 	if paintThingType != null:
 		var detectTerrainHeight = oDataClm.height[oDataClmPos.get_cell_clmpos(placeSubtile.x, placeSubtile.y)]
 		var newPos:Vector3 = Vector3(placeSubtile.x + 0.5, placeSubtile.y + 0.5, detectTerrainHeight)

Scenes/Selector.gd

@@ -303,8 +303,7 @@ func moved_to_new_subtile():
 
 	if mode == MODE_SUBTILE:
 		canPlace = true
-		if oUseSlabOwnerCheckBox.pressed == true and visible == true:
-			oSelection.paintOwnership = oDataOwnership.get_cellv_ownership(cursorTile)
+
 
 #func fadeOutWalls(delta):
 #	if Slabs.array[oSelection.cursorOverSlab][Slabs.SIDE_OF] == Slabs.SIDE_SLAB:

Scenes/Ui.gd

@@ -17,6 +17,8 @@ onready var oMapBrowser = Nodelist.list["oMapBrowser"]
 onready var oCamera3D = Nodelist.list["oCamera3D"]
 onready var oPlayer = Nodelist.list["oPlayer"]
 onready var o3DCameraInfo = Nodelist.list["o3DCameraInfo"]
+onready var oSelection = Nodelist.list["oSelection"]
+onready var oSelector = Nodelist.list["oSelector"]
 
 var FONT_SIZE_CR_LVL_BASE := 1.00 setget set_FONT_SIZE_CR_LVL_BASE
 var FONT_SIZE_CR_LVL_MAX := 8.00 setget set_FONT_SIZE_CR_LVL_MAX
@@ -69,32 +71,10 @@ func _unhandled_input(event):
 			mouseOnUi = true
 
 
-
-#func _process(delta):
-#	if 
-#	var a = Control.new()
-#	add_child(a)
-#	print(a.get_focus_owner())
-
-
-
 func update_theme_colour(val):
-	var col
-	match val:
-		0: col = HSV_8(0,40,60)#Color8(132,44,0,255)
-		1: col = HSV_8(279,30,66)#Color8(136,112,148,255)##
-		2: col = HSV_8(88,34,50)#col = Color8(52,96,4,255)
-		3: col = HSV_8(55,35,70)#col = Color8(188,156,0,255)
-		4: col = HSV_8(0,0,72)#col = Color8(180,160,124,255)
-		5: col = HSV_8(251,20,40)#col = Color8(64,62,72,255)
-		6: col = HSV_8(330,40,60)
-		7: col = HSV_8(0,0,2)
-		8: col = HSV_8(30,50,70)
-#	print(col.r8)
-#	print(col.g8)
-#	print(col.b8)
-	
-	windowStyleBoxFlat.set('border_color', col)
+	var col = Constants.windowTitleCol[val]
+	if windowStyleBoxFlat.get('border_color') != col:
+		windowStyleBoxFlat.set('border_color', col)
 
 func HSV_8(h,s,v):
 	return Color.from_hsv(h/359.0,s/100.0,v/100.0, 1.0)