@quenty/colorpalette 10.6.3-canary.500.0c613a3.0 → 10.7.0

package/CHANGELOG.md

@@ -3,12 +3,9 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
-## [10.6.3-canary.500.0c613a3.0](https://github.com/Quenty/NevermoreEngine/compare/@quenty/colorpalette@10.6.2...@quenty/colorpalette@10.6.3-canary.500.0c613a3.0) (2024-09-25)
+# [10.7.0](https://github.com/Quenty/NevermoreEngine/compare/@quenty/colorpalette@10.6.2...@quenty/colorpalette@10.7.0) (2024-09-25)
 
-
-### Bug Fixes
-
-* Ensure backwards compatibility ([97995a8](https://github.com/Quenty/NevermoreEngine/commit/97995a812c8d1d8489104b16e7ac443fbb2effec))
+**Note:** Version bump only for package @quenty/colorpalette
 
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@quenty/colorpalette",
-  "version": "10.6.3-canary.500.0c613a3.0",
+  "version": "10.7.0",
   "description": "Color palette system for Roblox",
   "keywords": [
     "Roblox",
@@ -28,18 +28,17 @@
     "access": "public"
   },
   "dependencies": {
-    "@quenty/baseobject": "10.4.1-canary.500.0c613a3.0",
-    "@quenty/blend": "12.5.3-canary.500.0c613a3.0",
-    "@quenty/color3utils": "11.5.3-canary.500.0c613a3.0",
-    "@quenty/colorpicker": "10.6.3-canary.500.0c613a3.0",
-    "@quenty/loader": "10.4.1-canary.500.0c613a3.0",
-    "@quenty/maid": "3.3.0",
-    "@quenty/math": "2.7.0",
-    "@quenty/observablecollection": "12.5.3-canary.500.0c613a3.0",
-    "@quenty/rx": "13.5.1-canary.500.0c613a3.0",
-    "@quenty/signal": "7.4.1-canary.500.0c613a3.0",
-    "@quenty/table": "3.5.0",
-    "@quenty/valueobject": "13.5.1-canary.500.0c613a3.0"
+    "@quenty/baseobject": "^10.5.0",
+    "@quenty/blend": "^12.6.0",
+    "@quenty/color3utils": "^11.6.0",
+    "@quenty/colorpicker": "^10.7.0",
+    "@quenty/loader": "^10.5.0",
+    "@quenty/maid": "^3.3.0",
+    "@quenty/math": "^2.7.0",
+    "@quenty/rx": "^13.6.0",
+    "@quenty/signal": "^7.5.0",
+    "@quenty/table": "^3.5.0",
+    "@quenty/valueobject": "^13.6.0"
   },
-  "gitHead": "0c613a3ae0b6ba6a4cda511f572220bfa951c70d"
+  "gitHead": "41715b15e2b48b2d22ff4f5277a8d4b7a0d32ef3"
 }

package/src/Shared/ColorPalette.lua

@@ -9,11 +9,13 @@ local ValueObject = require("ValueObject")
 local ColorSwatch = require("ColorSwatch")
 local ColorGradePalette = require("ColorGradePalette")
 local Rx = require("Rx")
+local Maid = require("Maid")
 local Blend = require("Blend")
 local Observable = require("Observable")
+local Signal = require("Signal")
+local Table = require("Table")
 local ColorGradeUtils = require("ColorGradeUtils")
 local LuvColor3Utils = require("LuvColor3Utils")
-local ObservableMap = require("ObservableMap")
 
 local ColorPalette = setmetatable({}, BaseObject)
 ColorPalette.ClassName = "ColorPalette"
@@ -23,20 +25,23 @@ function ColorPalette.new()
 	local self = setmetatable(BaseObject.new(), ColorPalette)
 
 	self._gradePalette = self._maid:Add(ColorGradePalette.new())
+	self._gradeMaid = self._maid:Add(Maid.new())
+	self._colorMaid = self._maid:Add(Maid.new())
+	self._vividMaid = self._maid:Add(Maid.new())
 
-	self._swatchMap = self._maid:Add(ObservableMap.new())
-	self._colorGradeMap = self._maid:Add(ObservableMap.new())
+	self._swatches = {}
 	self._colorValues = {}
+	self._colorGradeValues = {}
 	self._vividnessValues = {}
 
-	self.ColorSwatchAdded = assert(self._swatchMap.KeyAdded, "No KeyAdded") -- :Fire(name)
-	self.ColorGradeAdded = assert(self._colorGradeMap.KeyAdded, "No KeyAdded") -- :Fire(name) -- :Fire(name)
+	self.ColorSwatchAdded = self._maid:Add(Signal.new()) -- :Fire(name)
+	self.ColorGradeAdded = self._maid:Add(Signal.new()) -- :Fire(name)
 
 	return self
 end
 
 function ColorPalette:GetSwatchNames()
-	return self._swatchMap:GetKeyList()
+	return Table.keys(self._swatches)
 end
 
 function ColorPalette:ObserveSwatchNames()
@@ -47,20 +52,8 @@ function ColorPalette:ObserveSwatchNames()
 	})
 end
 
-function ColorPalette:ObserveSwatchNameList()
-	return self._swatchMap:ObserveKeyList()
-end
-
-function ColorPalette:ObserveSwatchNamesBrio()
-	return self._swatchMap:ObserveKeysBrio()
-end
-
 function ColorPalette:GetGradeNames()
-	return self._colorGradeMap:GetKeyList()
-end
-
-function ColorPalette:ObserveGradeNameList()
-	return self._colorGradeMap:ObserveKeyList()
+	return Table.keys(self._colorGradeValues)
 end
 
 function ColorPalette:ObserveGradeNames()
@@ -71,10 +64,6 @@ function ColorPalette:ObserveGradeNames()
 	})
 end
 
-function ColorPalette:ObserveGradeNamesBrio()
-	return self._colorGradeMap:ObserveKeysBrio()
-end
-
 function ColorPalette:GetColorValues()
 	return self._colorValues
 end
@@ -141,7 +130,7 @@ function ColorPalette:SetDefaultSurfaceName(surfaceName)
 end
 
 function ColorPalette:GetColorSwatch(colorName)
-	local swatch = self._swatchMap:Get(colorName)
+	local swatch = self._swatches[colorName]
 	if not swatch then
 		error(string.format("No swatch with name %q", colorName))
 	end
@@ -247,7 +236,7 @@ function ColorPalette:GetColorValue(colorName)
 end
 
 function ColorPalette:GetGradeValue(gradeName)
-	local gradeValue = self._colorGradeMap:Get(gradeName)
+	local gradeValue = self._colorGradeValues[gradeName]
 	if not gradeValue then
 		error(string.format("No grade with name %q", gradeName))
 	end
@@ -284,7 +273,7 @@ end
 function ColorPalette:GetSwatch(swatchName)
 	assert(type(swatchName) == "string", "Bad swatchName")
 
-	local swatch = self._swatchMap:Get(swatchName)
+	local swatch = self._swatches[swatchName]
 	if not swatch then
 		error(string.format("No swatch with name %q", swatchName))
 	end
@@ -292,10 +281,6 @@ function ColorPalette:GetSwatch(swatchName)
 	return swatch
 end
 
---[=[
-	@param colorName string
-	@param color Observable<Color3> | Color3
-]=]
 function ColorPalette:SetColor(colorName, color)
 	assert(type(colorName) == "string", "Bad colorName")
 
@@ -303,7 +288,7 @@ function ColorPalette:SetColor(colorName, color)
 		error(string.format("No color grade with name %q", colorName))
 	end
 
-	return self._colorValues[colorName]:Mount(color)
+	self._colorMaid[colorName] = self._colorValues[colorName]:Mount(color)
 end
 
 function ColorPalette:SetVividness(gradeName, vividness)
@@ -313,19 +298,18 @@ function ColorPalette:SetVividness(gradeName, vividness)
 		error(string.format("No vividness with name %q", gradeName))
 	end
 
-	return self._vividnessValues[gradeName]:Mount(vividness)
+	self._vividMaid[gradeName] = self._vividnessValues[gradeName]:Mount(vividness)
 end
 
 function ColorPalette:SetColorGrade(gradeName, grade)
 	assert(type(gradeName) == "string", "Bad colorName")
 	assert(grade, "Bad grade")
 
-	local gradeValue = self._colorGradeMap:Get(gradeName)
-	if not gradeValue then
+	if not self._colorGradeValues[gradeName] then
 		error(string.format("No color grade with name %q", gradeName))
 	end
 
-	return gradeValue:Mount(grade)
+	self._gradeMaid[gradeName] = self._colorGradeValues[gradeName]:Mount(grade)
 end
 
 
@@ -338,21 +322,26 @@ end
 function ColorPalette:DefineColorGrade(gradeName, gradeValue, vividnessValue)
 	assert(type(gradeName) == "string", "Bad gradeName")
 
-	if self._colorGradeMap:Get(gradeName) then
+	if self._colorGradeValues[gradeName] then
 		warn(string.format("[ColorPalette.DefineColorGrade] - Already defined grade of name %q", gradeName))
 		return
 	end
 
-	local colorGrade = self._maid:Add(ValueObject.new(0, "number"))
-	colorGrade:Mount(gradeValue or 0)
+	local colorGrade = ValueObject.new(0, "number")
+	self._maid:GiveTask(colorGrade)
+
+	local vividness = ValueObject.new(nil)
+	self._maid:GiveTask(vividness)
 
-	local vividness = self._maid:Add(ValueObject.new(nil))
-	vividness:Mount(vividnessValue)
+	self._gradePalette:Add(gradeName, colorGrade, vividness)
 
+	self._colorGradeValues[gradeName] = colorGrade
 	self._vividnessValues[gradeName] = vividness
 
-	self._gradePalette:Add(gradeName, colorGrade, vividness)
-	self._colorGradeMap:Set(gradeName, colorGrade)
+	self:SetVividness(gradeName, vividnessValue)
+	self:SetColorGrade(gradeName, gradeValue or 0)
+
+	self.ColorGradeAdded:Fire(gradeName)
 
 	return colorGrade
 end
@@ -360,17 +349,21 @@ end
 function ColorPalette:DefineColorSwatch(colorName, value)
 	assert(type(colorName) == "string", "Bad colorName")
 
-	if self._swatchMap:Get(colorName) then
+	if self._swatches[colorName] then
 		warn(string.format("[ColorPalette.DefineColorGrade] -Already defined color of name %q", colorName))
 		return
 	end
 
-	local colorValue = self._maid:Add(ValueObject.new(value or Color3.new(0, 0, 0), "Color3"))
-	local colorSwatch = self._maid:Add(ColorSwatch.new(colorValue))
+	local colorValue = ValueObject.new(value or Color3.new(0, 0, 0), "Color3")
+	self._maid:GiveTask(colorValue)
+
+	local colorSwatch = ColorSwatch.new(colorValue)
+	self._maid:GiveTask(colorSwatch)
 
 	self._colorValues[colorName] = colorValue
+	self._swatches[colorName] = colorSwatch
 
-	self._swatchMap:Set(colorName, colorSwatch)
+	self.ColorSwatchAdded:Fire(colorName)
 
 	return colorSwatch
 end

package/src/Shared/Font/FontPalette.lua

@@ -7,9 +7,10 @@
 local require = require(script.Parent.loader).load(script)
 
 local BaseObject = require("BaseObject")
+local Signal = require("Signal")
+local Table = require("Table")
 local Rx = require("Rx")
 local ValueObject = require("ValueObject")
-local ObservableSet = require("ObservableSet")
 
 local FontPalette = setmetatable({}, BaseObject)
 FontPalette.ClassName = "FontPalette"
@@ -27,9 +28,7 @@ function FontPalette.new()
 	self._fontFaces = {}
 	self._defaultFontMap = {} -- [name] = Enum.Font.?
 
-	self._fontKeys = self._maid:Add(ObservableSet.new())
-
-	self.FontAdded = assert(self._fontKeys.ItemAdded, "No ItemAdded") -- :Fire(name)
+	self.FontAdded = self._maid:Add(Signal.new()) -- :Fire(name)
 
 	return self
 end
@@ -40,12 +39,13 @@ end
 	@return { string }
 ]=]
 function FontPalette:GetFontNames()
-	return self._fontKeys:GetList()
+	return Table.keys(self._fonts)
 end
 
 --[=[
 	Observes all available font names as they are added starting with
 	existing fonts.
+
 	@return Observable<string>
 ]=]
 function FontPalette:ObserveFontNames()
@@ -61,16 +61,6 @@ function FontPalette:ObserveFontNames()
 	})
 end
 
---[=[
-	Observes all available font names as they are added starting with
-	existing fonts.
-
-	@return Observable<Brio<string>>
-]=]
-function FontPalette:ObserveFontNamesBrio()
-	return self._fontKeys:ObserveItemsBrio()
-end
-
 --[=[
 	Gets a font from name
 
@@ -191,8 +181,11 @@ function FontPalette:DefineFont(fontName, defaultFont)
 		error("Bad defaultFont")
 	end
 
-	local fontValue = self._maid:Add(ValueObject.new(defaultFontEnum))
-	local fontFaceValue = self._maid:Add(ValueObject.new(defaultFontFace))
+	local fontValue = ValueObject.new(defaultFontEnum)
+	self._maid:GiveTask(fontValue)
+
+	local fontFaceValue = ValueObject.new(defaultFontFace)
+	self._maid:GiveTask(fontFaceValue)
 
 	self._fonts[fontName] = fontValue
 	self._fontFaces[fontName] = fontFaceValue
@@ -211,7 +204,7 @@ function FontPalette:DefineFont(fontName, defaultFont)
 		end
 	end))
 
-	self._fontKeys:Add(fontName)
+	self.FontAdded:Fire(fontName)
 
 	return fontValue
 end