Update icon

Going to need a D10 merge request or patch to roll this in!

Added the contents of the patch in #4 as an MR. The MR needs testing and review. Moving this to needs review.

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++

Actually the #4 → doesn't work properly anymore on Drupal 10 with Admin Toolbar Tools enabled.

The Drupal icon comes then from tools.css and 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.

Patch #4 works for me using Drupal 10.3.1

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-9 class 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-10 classes are needed to work properly in Drupal 10.

liuyuanchao → changed the visibility of the branch 3163092-update-icon to hidden.

I add a new MR for the issue.
But I am wondering if it would be better to not support the Drupal8 and remove the drupal-8.svg.

Thanks liuyuanchao, your MR seems to work way better than the other recent patches.

In the era of Drupal 11, I agree with you on removing the Drupal 8 icon.