Modified background color for admin toolbar manage button

Created on 1 August 2024, 5 months ago
Updated 21 August 2024, 4 months ago

Problem/Motivation

With Drupal 10.3.1, the Admin toolbar's manage button background remains black with black text, but the background needs to be white.

Steps to reproduce

Use Drupal 10.3.1 without the Adminimal Admin Toolbarโ€”the text and the background are inverses of each other.
Enable the Adminimal Admin Toolbar, and the opened manage button will be black with black text.

Proposed resolution

I deleted two lines of CSS: the .is-active and :focus class background overrides.

๐Ÿ› Bug report
Status

RTBC

Version

2.0

Component

Code

Created by

๐Ÿ‡ญ๐Ÿ‡บHungary pzajacz

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @pzajacz
  • ๐Ÿ‡ญ๐Ÿ‡บHungary pzajacz

    Here is the patch with the removed CSS lines.

  • First commit to issue fork.
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States amanire

    The patch in #2 changes the way that adminimal admin toolbar has previously been styled in a jarring way by allowing a light background color scheme pass through on the active top level menu items.

    I've attached a patch that restores the styling to match previous dark background version of the adminimal admin toolbar.

  • Status changed to Needs review 5 months ago
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States amanire

    I put the patch from comment #4 into a MR.

  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia snehal-chibde

    hello @amanire , I have checked your MR on drupal 10.3.1.
    adminimal_admin_toolbar version 2.0.x.

    The changes are working as expected and now the manage button is seen with white text and white hamburger icon and visible correctly on the admin toolbar.

    Added before and after screenshots for reference.
    RTBC++

  • Status changed to RTBC 4 months ago
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States amanire

    Nice! I've been using the change on prod. I think we can update the issue status.

  • ๐Ÿ‡ซ๐Ÿ‡ทFrance Cedrick...

    Hello,
    I used the patch 4. It works well on desktop but i still have trouble with the responsive.
    So i implemented some background color on responsive mode.
    That's better.

    I share with you my patch

  • ๐Ÿ‡ซ๐Ÿ‡ทFrance Cedrick...

    Hello,
    I used the patch 4. It works well on desktop but i still have trouble with the responsive.
    So i implemented some background color on responsive mode.
    That's better.

    I share with you my patch

Production build 0.71.5 2024