Problem/Motivation

In #3576683: [meta] UX refresh, system_breadcrumb_block has been added to the exclude list of Block Library Panel for the default profile alongside:

  • system_menu_block:admin
  • system_menu_block:devel
  • system_clear_cache_block
  • local_actions_block

The other blocks are targeting admin pages, so it is understandable they are out by default until #3575891: Allow administration page display is merged, but system_breadcrumb_block is different because it is expected to both admin and front theme.

What is the reason? Something wrong with this block in particular?

Proposed resolution

Remove system_breadcrumb_block from the exclude list.

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

pdureau created an issue. See original summary.

pdureau’s picture

Assigned: pdureau » mogtofu33
Status: Active » Needs review

Is it the opportunity of changing the storage of exclude_id:

  block_library:
    exclude_id: "system_menu_block:admin\r\nsystem_menu_block:devel\r\nsystem_clear_cache_block\r\nlocal_actions_block\r\n"

from a string to a proper list, like what we did for the Pattern Preset sources in #3534190: Pattern presets storage and dependencies?

mogtofu33 made their first commit to this issue’s fork.

mogtofu33’s picture

Assigned: mogtofu33 » Unassigned
Status: Needs review » Fixed

It was a mistake, added the exclude_id change as a child issue.

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.

  • mogtofu33 committed 243ebd6c on 1.0.x authored by pdureau
    task: #3615464 Allow system_breadcrumb_block by default
    
    By: pdureau
    

Status: Fixed » Closed (fixed)

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