Scope precedence: buffer > global

[lsvmello]

Many mini features can be disabled using the same flag pattern by using vim.g.<minifeature>_disable or vim.b.<minifeature>_disable> depending on the scope (Global or buffer-wise). When any scope is set to true the feature is disabled.

I would like to suggest that buffer variables takes precedence over global variables.

For example:

Global	Buffer	Current behavior	Suggested behavior
nil	nil	enabled	enabled
nil	true	disabled	disabled
nil	false	enabled	enabled
true	nil	disabled	disabled
true	true	disabled	disabled
true	false	disabled	enabled
false	nil	enabled	enabled
false	false	enabled	enabled
false	true	disabled	disabled

This could easily be done using vim.F.if_nil

H.is_disabled = function() return vim.F.if_nil(vim.b.miniindentscope_disable, vim.g.miniindentscope_disable, false) end

I suggested this, because I particularly would like indentscope to be disabled by default, and enable only on buffers that I want, so I disable it globally after setup() and would change the buffer flag on the buffers I want using a command or keymap.

[echasnovski]

Thanks for the suggestion!

This was already discussed in #1552 with the following resolution and motivation behind it.

[lsvmello]

Oh, you are too fast hahaha I accidently posted without properly finishing it.

[echasnovski]

Oh, you are too fast hahaha I accidently posted without properly finishing it.

You gave the necessary title, that was enough 😅

The buffer-local variable having three valuable states (nil, false, true) instead of two (false-ish nil+false and true) is one of the main reasons why I am hesitant to implement this kind of behavior (as mentioned in linked comment).

I suggested this, because I particularly would like indentscope to be disabled by default, and enable only on buffers that I want, so I disable it globally after setup() and would change the buffer flag on the buffers I want using a command or keymap.

The suggested way to achieve this is to automatically set buffer-local variable to true and set it to false when want to enable it.