Customizing header logo size doesn't work

[matthias.waller@…]

The width and height settings in trac.ini are noneffective.

Look in share/trac/templates/layout.html and add

width="${chrome.logo.width}" height="${chrome.logo.height}"

to the header image tag (~ line 39).

[Christian Boos]

Ok, IIRC, care must be taken to keep those value to None when not set (we used to have some hard-coded defaults), to be checked.

[osimons <simon-code@…>]

This works:

trac/templates/layout.html

@@ -39 +39 @@
     <div id="banner">
       <div id="header" py:choose="">
         <a py:when="chrome.logo.src" id="logo" href="${chrome.logo.link}"><img
-          src="${chrome.logo.src}" alt="${chrome.logo.alt}" /></a>
+          src="${chrome.logo.src}" alt="${chrome.logo.alt}"
+          height="${chrome.logo.height or None}" width="${chrome.logo.width or None}" /></a>
         <h1 py:otherwise=""><a href="${chrome.logo.link}">${project.name}</a></h1>
       </div>
       <form py:if="'SEARCH_VIEW' in perm" id="search"

Setting values to 0 or -1 drops the attributes.

I can put it in if/when access is OK'ed.

[osimons]

Patch applied in [6225].