Add Hide action to the section actions

✨ Add visibility control conditions to blocks within Layout Builder Needs work

Ahmad Abbad → made their first commit to this issue’s fork.

Since Varbase is using the lb_ux module
I added a patch to work with a core ✨ Add visibility control conditions to blocks within Layout Builder Needs work patch (10.0X).

before applying this patch you should use the mentioned patch

I have rewritten patch #2. I will improve the functionality in the future, but for now, I need to continue using the patch to update the module.

Added access false to remove the html of the hidden section

Adding a ➖ UX/UI designer responsibilities

Add a tooltip to the hide section icon.

Add a tooltip to the hide section icon.

Reviewed the MR for this issue.
And then had a look at the code for ✨ Add visibility control conditions to blocks within Layout Builder Needs work

In Drupal Core MR, they are doing this in the Block level
https://git.drupalcode.org/project/drupal/-/merge_requests/3627/diffs#18...

In Varbase Layout Builder MR

The varbase_layout_builder.visibility_section route:
/layout_builder/visibility/section/{section_storage_type}/{section_storage}/{delta}
Has no {uuid} for the section

But in the MR for Drupal Core

layout_builder.add_visibility
/layout_builder/visibility/block/{section_storage_type}/{section_storage}/{delta}/{uuid}/{plugin_id}

layout_builder.visibility
/layout_builder/visibility/block/{section_storage_type}/{section_storage}/{delta}/{uuid}

layout_builder.delete_visibility
/layout_builder/visibility/block/{section_storage_type}/{section_storage}/{delta}/{uuid}/{plugin_id}/delete

Better to add the {uuid} for the section in our case as the {delta} may change at any time. but the uuid for the section will be synced with section.

Had a one more round of testing.
I have a sound thought about the use {delta}
Tested with reorder of sections, and it's working well too.
No longer needing the uuid, it is needed for blocks in the Drupal Core patch.
Moving forward with the merge

I enhanced this patch to also work with blocks, enabling us to hide sections and blocks.

Thanks, Mutasim.

to better review what are you addtions on blocks.

1. Create a new branch from the current issue fork ( with latest merged work )
2. Commit the patch to it.
3. Create new Merge Requiest fro the branch

Thanks Mutasim and Rajab. I believe we should open a new issue specifically for hiding blocks feature as it has been committed and is close to completion.

I agree with that.
In new issues.
Let us also check with our Product Owner and the Design team, to find a better look for
Section Action and Block Actions with hints/tooltips

Okay, I will create a new issue for the blocks and provide a patch for it.

No further patches are needed for Varbase 10
Removed form the description

✅ Released varbase_layout_builder-10.1.0 →

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

✅ Release varbase-10.0.0 →