Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.When viewing the site as a non-logged in user then the umami orange favicon is used
But when you log in and then edit anything the standard drupal blue favicon is used
As a new user I find this very strange and it makes it hard to find the correct tab in the browser
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | umami-favicon-3028608.patch | 605 bytes | danharper |
| #2 | Seven___Drupal.png | 242.32 KB | Eli-T |
Comments
Comment #2
Eli-TSelection of the favicon is the responsibility of the theme.
Umami uses the Seven theme, and is configured use the favicon supplied by that theme, the Druplicon. So this is not a bug.
However, it would be good if Umami still did use the Umami favicon when performing administration. This can be achieved in the settings for Seven as follows:
Comment #3
danharper CreditAttribution: danharper as a volunteer and at hrpr commentedI configured the seven theme and exported the config into the config/install directory but it wouldn't install so I had to use global config instead and add system.theme.global.yml into the config/install directory.
Comment #4
markconroy CreditAttribution: markconroy as a volunteer and at Annertech commentedUnassigning so someone can review it, or complete follow-on work that may be needed.
Comment #5
markconroy CreditAttribution: markconroy as a volunteer and at Annertech commentedThis works a treat. A lovely, simple, helpful improvement. Thanks folks.
Comment #6
alexpottCrediting @Not Real for filing the issue and @Eli-T and @markconroy for issue review and management.
Committed and pushed c88201f6ab to 8.7.x and 7888777609 to 8.6.x. Thanks!
As an Umami config improvement backported to 8.6.x
Comment #9
pawandubey CreditAttribution: pawandubey as a volunteer and at TATA Consultancy Services commentedI have tested the patch#3 and its working as expected.
Comment #10
Eli-TThanks @pawandubey! However, this has already been committed so putting back to fixed :)