Toolbar menu icons not displayed when active

This is an issue in D10.

1. E.g. the adminimal_theme/css/adminimal.css at line 759 looks for icons at given path e.g. /core/themes/stable/images/core/icons/787878/puzzlepiece.svg and it gives 404.
2. It's because there is no /stable/ because it has been removed https://www.drupal.org/node/3309392 →
3. The right path to the icon is e.g. /core/misc/icons/787878/puzzlepiece.svg

Created patch for toolbar menu icons issue.
Kindly review.

Regards!

Patch works. Thanks.

Noticed "page not found" errors with Adminimal trying to use the core stable paths for the icons. #9 patch fixed it nicely

This issue is related to Drupal 10.
Please check the below steps to reproduce this issue

1. Install Drupal 10.
2. Add "Adminimal" as an administration theme.
3. Now click on "Structure", for the active menu items, the icons of the toolbar aren't displaying.

It's because there is no stable theme existing in the core folder.

So we need to update the background image path instead of "/core/themes/stable/images/core/icons/787878/file.svg" need to update "/core/misc/icons/787878/file.svg".

Please check the below code with the updated path.

/* Main menu icons. */
.adminimal .toolbar-icon-system-admin-content:active:before,
.adminimal .toolbar-icon-system-admin-content.is-active:before {
  background-image: url(/core/misc/icons/787878/file.svg);
}

.adminimal .toolbar-icon-system-admin-structure:active:before,
.adminimal .toolbar-icon-system-admin-structure.is-active:before {
  background-image: url(/core/misc/icons/787878/orgchart.svg);
}

.adminimal .toolbar-icon-system-themes-page:active:before,
.adminimal .toolbar-icon-system-themes-page.is-active:before {
  background-image: url(/core/misc/icons/787878/paintbrush.svg);
}

.adminimal .toolbar-icon-entity-user-collection:active:before,
.adminimal .toolbar-icon-entity-user-collection.is-active:before {
  background-image: url(/core/misc/icons/787878/people.svg);
}

.adminimal .toolbar-icon-system-modules-list:active:before,
.adminimal .toolbar-icon-system-modules-list.is-active:before {
  background-image: url(/core/misc/icons/787878/puzzlepiece.svg);
}

.adminimal .toolbar-icon-system-admin-config:active:before,
.adminimal .toolbar-icon-system-admin-config.is-active:before {
  background-image: url(/core/misc/icons/787878/wrench.svg);
}

.adminimal .toolbar-icon-system-admin-reports:active:before,
.adminimal .toolbar-icon-system-admin-reports.is-active:before {
  background-image: url(/core/misc/icons/787878/barchart.svg);
}

.adminimal .toolbar-icon-help-main:active:before,
.adminimal .toolbar-icon-help-main.is-active:before {
  background-image: url(/core/misc/icons/787878/questionmark-disc.svg);
}

Please check the updated patch. I have added interdiff file to compare #9 patch.

+Needs review

Patch from #12 🐛 Toolbar menu icons not displayed when active RTBC works nicely and should be preferred over the patch from #9 🐛 Toolbar menu icons not displayed when active RTBC : the latter uses data URIs containing icons which seems unnecessary to me given that the icons reside in /core/misc/icons already. Moving to RTBC.

Side node: the patch from #3 fixes some CSS coding standard violations unrelated to this issue (while also introducing some new violations). These should be addressed separately.

Patch from #12 also works for us, thanks.

Yep, patch #12 also working this side.

Drupal core: 10.1.5
Adminimal Theme: 8.x-1.7

Another thumbs up for #12.

Patch on #12 works like a charm!

There is also a reference to stable within the info.yml file. Here is a patch to map core's system theme to the (now contrib) seven theme. This should be applied alongside #12.

Hi everyone, I made a patch that merge both the #12 one and the #18 one. Works on Drupal 10.3.1 and adminimal_theme 8.x-1.7.

#19 works. Thanks!