Problem/Motivation

Having the Layout Builder Blocks works in both Back-end and Front-End theme.

Proposed resolution

  • Limit Layout Builder Blocks to the front-end theme
  • Check on the current active theme if it is not the default Back-end theme
  • Supporting Layout Builder Modal

Remaining tasks

  • File an issue
  • Patch/MR
  • Review
  • Test

User interface changes

  • N/A

API changes

  • N/A

Data model changes

  • N/A
Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

Rajab Natshah created an issue. See original summary.

rajab natshah’s picture

StatusFileSize
new814 bytes

rajab natshah’s picture

Assigned: rajab natshah » Unassigned
Status: Active » Needs review

Having a quick if check condition.
This could be managed for all blocks in all active theme ( In configurations )

rajab natshah’s picture

StatusFileSize
new815 bytes
rajab natshah’s picture

rajab natshah’s picture

Assigned: Unassigned » rajab natshah
Status: Needs review » Active
rajab natshah’s picture

Issue summary: View changes
Status: Active » Needs review

Supporting Layout Builder Modal

rajab natshah’s picture

Assigned: rajab natshah » Unassigned
rajab natshah’s picture

StatusFileSize
new9.9 KB
rajab natshah’s picture

StatusFileSize
new9.15 KB
rajab natshah’s picture

Assigned: Unassigned » rajab natshah
Status: Needs review » Active
rajab natshah’s picture

StatusFileSize
new9.46 KB
rajab natshah’s picture

rajab natshah’s picture

Assigned: rajab natshah » Unassigned
Status: Active » Needs work

Needs a config settings to have a better control, for where this module works.
For example not to work with a Dashboard with layout builder under Claro, Gin, or others

The settings could work like apply for themes, except checked not to use under some themes ( Maybe better logic )

rajab natshah’s picture

Title: Limit Layout Builder Blocks to the front-end theme » Limit Layout Builder Blocks not to work in the dashboards route
rajab natshah’s picture

Attached the layout_builder_blocks--2024-08-14--3349066--mr-5.patch static file. ( from MR5 )

To be used with Composer Patches.
Not planing to commit this change into the module.

---
A better fix could be having a config with limited routes ( which this module should works in )
and have the dashboard route in the list..

n.ghunaim’s picture

Include Dashboard module routes to prevent All BLB or VLB from showing up

rajab natshah’s picture

rajab natshah’s picture

Status: Needs work » Needs review

rajab natshah changed the visibility of the branch 3349066-limit-layout-builder to hidden.

  • rajab natshah committed 29550c19 on 1.0.x
    feat: #3349066 Limit Layout Builder Blocks not to work in the dashboards...
rajab natshah’s picture

Assigned: Unassigned » Amira Dowidar
Issue tags: +layout_builder_blocks-1.1.3
rajab natshah’s picture

rajab natshah’s picture

Assigned: aaronchristian » Unassigned
Status: Needs review » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

rajab natshah’s picture

Version: 1.0.x-dev » 1.1.x-dev

  • rajab natshah committed 1e5dcbf3 on 1.1.x
    feat: #3349066 Limit Layout Builder Blocks not to work in the dashboards...
rajab natshah’s picture

Status: Fixed » Closed (fixed)

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