Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.If the Bartik color preview page is visited using a language prefix in the path (e.g. /fa/admin/appearance/settings/bartik), the site logo isn't loaded in the preview section, because the logo path is hard-coded in the html as a relative path, which fails with the extra language prefix.
There is some javascript in the related color/preview.js in Bartik to replace the correct path for the logo, but it doesn't seem to work anymore. I don't have much knowledge in this part, but I suspect that #2160603: Remove drupal_add_js() from Bartik broke it, becuase the required setting isn't loaded into the page anymore.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | bartik-preview-screen.png | 36.26 KB | herom |
Comments
Comment #1
herom commentedHere is a screenshot of the broken logo, and hard-coded path.
Comment #2
gábor hojtsyIt should still be possible to add JS settings to the page with #attached. Therefore making novice.
Comment #3
balagan commentedComment #4
balagan commenteddrupalSettings.color.logo is undefined on the page, though in the color.inc it is set with #attached.
Comment #5
balagan commentedThis is a duplicate issue for the following: https://www.drupal.org/node/2396983
The patch in comment #1 solves this.
Comment #6
balagan commentedComment #7
gábor hojtsy#2396983: Header Logo with Bartik won't change in settings preview is resolved and committed so this should be resolved :) Yay!