fix(config): options.darken.sidebars.enabled

Problem: `options.darken.sidebars.enabled` and
         `options.darken.sidebars.enable` both exist, but it is only one
         option. The default in `config.lua` is `enable`, but it is
         referred to as `enabled` as well (both in code and docs). It
         used to be `enable` but was renamed to `enabled`, so it is
         technically supposed to be `options.darken.sidebars.enabled` at
         this time (as of this commit's parent).

Solution: Deprecate and revert `options.darken.sidebars.enabled` back to
          `options.darken.sidebars.enable`. This brings it back in-line
          with the changelog and runtime deprecation checks. `enable` is
          also more consistent as this is what is used in other parts
          of the config (e.g. config for `modules`).

Problem: In a couple of places, `options.darken.sidebars` is checked for
         truthiness. This seems to be a typo of
         `options.darken.sidebars.enable` which is a boolean (unlike the
         former, which is a table by default and also documented as a
         table). Tables are always truthy.

Solution: Use `options.darken.sidebars.enable` instead of
          `options.darken.sidebars` in conditionals.

Reverts: fff3e20, 806903c
Fixes: #306

Author: tmillr

CHANGELOG.md

@@ -7,6 +7,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [unreleased]
 
+### Configuration Deprecation and Migration
+
+-   Reverted/renamed `options.darken.sidebars.enabled` back to `options.darken.sidebars.enable` (see #343)
+
 ### What's New?
 
 -   Added new highlight groups for mini.nvim (#333 by @echasnovski)
@@ -18,10 +22,11 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Issues Fix
 
--   Fixed `punctuation.delimiter` treesitter group nearly invisible (#329 fixed by #331)
+-   Fixed `punctuation.delimiter` treesitter group nearly invisible (#329 fixed-by #331)
 -   Closed #305 (no longer valid, fixed)
 -   Closed #292 (no longer valid, fixed)
 -   fix(config): `options.darken.floats` is not used (#345)
+-   Fixed sidebar is not darkened when configured to do so (affects neo-tree, etc.) (#306 fixed-by #343)
 
 ## [v1.0.2] - 03 May 2023
 

Makefile

@@ -40,7 +40,7 @@ test: $(PLENARY_DIR)
 		--headless \
 		--noplugin \
 		-u test/minimal_init.vim \
-		-c "PlenaryBustedDirectory $(TEST_DIR) { minimal_init = './test/minimal_init.vim' }"
+		-c "PlenaryBustedDirectory $(TEST_DIR) { minimal_init = './test/minimal_init.vim', sequential = true }"
 
 $(PLENARY_DIR):
 	git clone --depth=1 --no-single-branch $(PLENARY_URL) $(PLENARY_DIR)

README.md

@@ -283,7 +283,7 @@ require('github-theme').setup({
     darken = {                 -- Darken floating windows and sidebar-like windows
       floats = true,
       sidebars = {
-        enabled = true,
+        enable = true,
         list = {},             -- Apply dark background to specific windows
       },
     },

Usage.md

@@ -249,8 +249,8 @@ Another setting is for sidebars, which is configured in the `sidebars` sub-table
 
 #### options.darken.sidebars {table}
 
-The `sidebars` sub table of `darken` contains settings for sidebar-like windows. It has two configurations: `enabled` is used to assign a darker background
-to the listed windows, and `list` specifies the windows to be included in the list.
+The `sidebars` sub table of `darken` contains settings for sidebar-like windows. It has two configurations: `enable` is used to assign a darker background
+to the listed windows, and `list` specifies the windows to be darkened.
 
 Example:
 
@@ -259,7 +259,7 @@ local options = {
   darken = {
     floats = false,
     sidebars = {
-      enabled = true,
+      enable = true,
       list = {'qf', 'netrw'} -- default is {}
     }
   }

doc/github-nvim-theme.txt

@@ -331,9 +331,9 @@ sub-table.
 options.darken.sidebars {table}    *github-nvim-theme-options.darken.sidebars*
 
 The `sidebars` sub table of `darken` contains settings for sidebar-like
-windows. It has two configurations: `enabled` is used to assign a darker
+windows. It has two configurations: `enable` is used to assign a darker
 background to the listed windows, and `list` specifies the windows to be
-included in the list.
+darkened.
 
 Example:
 
@@ -342,7 +342,7 @@ Example:
       darken = {
         floats = false,
         sidebars = {
-          enabled = true,
+          enable = true,
           list = {'qf', 'netrw'} -- default is {}
         }
       }

lua/github-theme/group/editor.lua

@@ -3,7 +3,7 @@ local C = require('github-theme.lib.color')
 local M = {}
 
 function M.get(spec, config)
-  local dark_sb = config.darken.sidebars.enabled
+  local dark_sb = config.darken.sidebars.enable
   local hide_eof = config.hide_end_of_buffer
   local inactive = config.dim_inactive
   local inv = config.inverse

lua/github-theme/group/modules/neotree.lua

@@ -7,7 +7,7 @@ local M = {}
 
 function M.get(spec, config, opts)
   local hide_eof = config.hide_end_of_buffer
-  local dark_sb = config.darken.sidebars
+  local dark_sb = config.darken.sidebars.enable
   local c = spec.palette
 
   local function blend(color, a)

lua/github-theme/group/modules/nvimtree.lua

@@ -4,7 +4,7 @@ local M = {}
 
 function M.get(spec, config, opts)
   local hide_eof = config.hide_end_of_buffer
-  local dark_sb = config.darken.sidebars
+  local dark_sb = config.darken.sidebars.enable
   local c = spec.palette
 
   -- stylua: ignore

lua/github-theme/util/deprecation.lua

@@ -157,6 +157,16 @@ M.check_deprecation = function(opts)
       ' for more info.'
     )
   end
+
+  if (((opts.options or {}).darken or {}).sidebars or {}).enabled ~= nil then
+    dep.write(
+      '  ',
+      { 'options.darken.sidebars.enabled', 'WarningMsg' },
+      ' has been renamed/reverted back to ',
+      { 'options.darken.sidebars.enable', 'WarningMsg' }
+    )
+  end
+
   M.checked_deprecation = true
 end
 

test/github-theme/config/darken_spec.lua

@@ -3,28 +3,33 @@ local t_util = require('github-theme._test.util')
 local C = require('github-theme.lib.color')
 local api = vim.api
 
-if not api.nvim_get_hl then
+if
+  (vim.fn.has('nvim-0.10.0') == false or vim.fn.has('nvim-0.10.0') == 0)
+  and not api.nvim_get_hl
+then
   return
 end
 
 describe('config > options > darken', function()
-  before_each(function()
+  after_each(function()
+    -- Could reset more than just this, but this alone seems to work now that the
+    -- compiler is fixed.
     require('github-theme.config').reset()
   end)
 
   describe('> floats', function()
     for _, variant in ipairs(require('github-theme.palette').themes) do
       -- TODO: see #324
-      local _it = variant:find('high[-_]*contrast') and pending or it
+      local it_or_pending = variant:find('high[-_]*contrast') and pending or it
 
-      _it(('should be enabled by default (%s)'):format(variant), function()
+      it_or_pending(('should be enabled by default (%s)'):format(variant), function()
         require('github-theme').setup()
         vim.cmd.colorscheme({ args = { variant } })
         local normal_float = t_util.get_hl('NormalFloat')
         local normal = t_util.get_hl('Normal')
 
         assert.is_true(require('github-theme.config').options.darken.floats)
-        assert.are.not_equal(normal_float.bg, normal.bg)
+        assert.are.not_equal(normal.bg, normal_float.bg)
         assert(
           C(('#%x'):format(normal_float.bg)):luminance()
             < C(('#%x'):format(normal.bg)):luminance(),
@@ -40,10 +45,60 @@ describe('config > options > darken', function()
         vim.cmd.colorscheme({ args = { variant } })
         local normal_float = t_util.get_hl('NormalFloat')
         local normal = t_util.get_hl('Normal')
-
         assert.is_false(require('github-theme.config').options.darken.floats)
-        assert.are.equal(normal_float.bg, normal.bg)
+        assert.are.equal(normal.bg, normal_float.bg)
       end)
     end
   end)
+
+  describe('> sidebars', function()
+    describe('> enable', function()
+      for _, variant in ipairs(require('github-theme.palette').themes) do
+        -- TODO: see #324
+        local it_or_pending = variant:find('high[-_]*contrast') and pending or it
+
+        it_or_pending(('should be enabled by default (%s)'):format(variant), function()
+          require('github-theme').setup()
+          vim.cmd.colorscheme({ args = { variant } })
+          local normal_sb = t_util.get_hl('NormalSB')
+          local normal = t_util.get_hl('Normal')
+
+          assert.is_true(require('github-theme.config').options.darken.sidebars.enable)
+          assert.are.not_equal(normal.bg, normal_sb.bg)
+          assert(
+            C(('#%x'):format(normal_sb.bg)):luminance()
+              < C(('#%x'):format(normal.bg)):luminance(),
+            ('expected `bg` of `NormalFloat` (#%x) to be darker than `bg` of `Normal` (#%x)'):format(
+              normal_sb.bg,
+              normal.bg
+            )
+          )
+        end)
+
+        it(('should be disabled when set to `false` (%s)'):format(variant), function()
+          require('github-theme').setup({
+            options = { darken = { sidebars = { enable = false } } },
+          })
+
+          vim.cmd.colorscheme({ args = { variant } })
+          local normal_sb = t_util.get_hl('NormalSB')
+          local normal = t_util.get_hl('Normal')
+          assert.is_false(require('github-theme.config').options.darken.sidebars.enable)
+          assert.are.equal(normal.bg, normal_sb.bg)
+        end)
+
+        it(('should also accept `enabled` (%s)'):format(variant), function()
+          require('github-theme').setup({
+            options = { darken = { sidebars = { enabled = false } } },
+          })
+
+          vim.cmd.colorscheme({ args = { variant } })
+          local normal_sb = t_util.get_hl('NormalSB')
+          local normal = t_util.get_hl('Normal')
+          assert.is_false(require('github-theme.config').options.darken.sidebars.enable)
+          assert.are.equal(normal.bg, normal_sb.bg)
+        end)
+      end
+    end)
+  end)
 end)