Fixes and utilities (#107)

* Fix(palette): Extend if required before theme is loaded

* Ref(utils): Reduce reliance on strings

Squash me

* Fix: Transparent background for floating border

See #48

lua/nordic/colors/init.lua

@@ -2,7 +2,11 @@ local U = require 'nordic.utils'
 local O = require('nordic.config').options
 local C = require 'nordic.colors.nordic'
 
+C.extended = false
+
 function C.extend_palette()
+    C.extended = true
+
     -- Modify the palette before generating colors.
     C = O.on_palette(C)
 
@@ -56,7 +60,7 @@ function C.extend_palette()
     C.fg_popup_border = C.border_fg
 
     -- Floating windows
-    C.bg_float = C.black1
+    C.bg_float = (O.transparent_bg and C.none) or C.black1
     C.fg_float = C.fg
     C.bg_float_border = C.bg_float
     C.fg_float_border = C.border_fg
@@ -90,4 +94,10 @@ function C.extend_palette()
     end
 end
 
+-- Sometimes the palette is required before the theme has been loaded,
+-- so we need to extend the palette in those cases.
+if not C.extended then
+    C.extend_palette()
+end
+
 return C

lua/nordic/utils.lua

@@ -11,6 +11,14 @@ function M.highlight(table)
     end
 end
 
+function M.is_none(string)
+    return string == 'NONE' or string == 'none'
+end
+
+function M.none()
+    return 'NONE'
+end
+
 function M.merge(table1, table2)
     if table1 == table2 == nil then return {} end
     if table1 == nil then
@@ -107,7 +115,7 @@ end
 
 -- Adapted from @folke/tokyonight.nvim.
 function M.blend(foreground, background, alpha)
-    if foreground == 'NONE' or background == 'NONE' then return 'NONE' end
+    if M.is_none(foreground) or M.is_none(background) then return M.none() end
 
     local fg = { M.hex_to_rgb(foreground) }
     local bg = { M.hex_to_rgb(background) }