Closed (fixed)
Project:
Drupal core
Version:
8.7.x-dev
Component:
Umami demo
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
26 Jan 2019 at 11:25 UTC
Updated:
10 Feb 2019 at 11:49 UTC
Jump to comment: Most recent, Most recent file
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 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 commentedUnassigning so someone can review it, or complete follow-on work that may be needed.
Comment #5
markconroy 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 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 :)