Block should only display if "display a table of contents" checkbox is checked

ilya.no → made their first commit to this issue’s fork.

I've created MR with this feature. Initially I tried to use hook_block_access in toc_js_per_node.module file, but it doesn't work for node pages, rendered with layout builder → , so I added function blockAccess and it works fine.

This feature should be configurable at the block level (default disabled to respect current behavior) with a checkbox added on the configuration form (support_toc_js_per_node for example) if the submodule is enabled.

Also
$value = $node->get('toc_js_active')->first()->get('value')->getValue();

could be simplified
$value = $node->get('toc_js_active')->value;

Implemented the code entirely in the toc_js_per_node for better code modularity and to avoid redundant configurations if the site doesn't make use of the per node behaviour.

As well as supporting two different use cases with the field:

1. Hide the block if "display a table of contents" is check (since it might be rendered using the Node view display rather than the block).
2. Hide/show exclusively using the value of "display a table of contents". So that it acts as a way of opting in/out of the rendering

Updated the logic for the "hide" behaviour, such that if there's been a value set on the content in some way, that should take precedence over the block settings, and be always hidden since its being rendered on the Node display level.

If we need other variations of logic, then we can always add more options to the list.

FYI I think it has been implemented in the new 3.0.x-dev version.

Automatically closed - issue fixed for 2 weeks with no activity.