Code review comment for lp://staging/~razamatan/loggerhead/favicon

Revision history for this message
Martin Albisetti (beuno) wrote :

Yay favicons!
One question:

10 + <link rel="shortcut icon"
11 + tal:attributes="href python:branch.static_url('/static/images/favicon.ico')" />

and

23 + <link rel="shortcut icon"
24 + tal:attributes="href python:branch.static_url('/static/images/favicon.ico')" />

Shouldn't these be .png?

review: Needs Information

« Back to merge proposal