- 🇺🇸United States energee
Going to need a D10 merge request or patch to roll this in!
- First commit to issue fork.
- Status changed to Needs review
8 months ago 2:31pm 28 June 2024 - 🇺🇸United States johnpicozzi Providence, RI
Added the contents of the patch in #4 as an MR. The MR needs testing and review. Moving this to needs review.
- Status changed to RTBC
6 months ago 7:55am 21 August 2024 - 🇮🇳India snehal-chibde
hello @johnpicozzi, I have tested the MR on drupal 10.3.1 version.
adminimal_admin_toolbar version 2.0.x.The changes are working as expected and now can see the updated icon of Drupal.
Added before and after screenshots for reference.
RTBC++ - Status changed to Needs work
about 1 month ago 11:39pm 16 January 2025 - 🇫🇷France alxgl
Actually the #4 → doesn't work properly anymore on Drupal 10 with Admin Toolbar Tools enabled.
The Drupal icon comes then from
tools.cssand it's now blue (instead of white in the original patch) and not well sized, because it's supposed to fit with Admin Toolbar layout.It's clearly visible in #17 📌 Update icon Needs work screens.
- 🇺🇸United States avattikonda Boston
Patch #4 works for me using Drupal 10.3.1
- 🇫🇷France alxgl
Like I said, it "works" if you're just expecting not to have a Drupal 8 logo but it's not good enough.
The patch is targeting a
.toolbar-icon-9class which does not exist anymore. The logo you see is not from the Adminimal Admin Toolbar module. It's a fallback from Admin Toolbar Tools.A new
drupal-10.svg(even just a copy of the drupal-9 version) and new.toolbar-icon-10classes are needed to work properly in Drupal 10.