Needs review
Project:
Touch
Version:
7.x-1.7
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
18 Sep 2012 at 09:31 UTC
Updated:
19 Sep 2012 at 13:46 UTC
Jump to comment: Most recent file
in page.tpl.php I found stuff like this
<img src="<?php global $base_url; echo $base_url.'/'.$directory; ?>/images/rss.png" alt="RSS"/>
That should be replaced with:
<img src="<?php echo drupal_get_path('theme', 'touch'); ?>/images/rss.png" alt="RSS"/>
Like this its also easier for subtheme authors who want to implement page.tpl.php without copy the icons
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | touch-use_drupal_get_path-1787606-4.patch | 1.89 KB | yannickoo |
| #1 | touch-use_drupal_get_path-1787606-1.patch | 1.89 KB | yannickoo |
Comments
Comment #1
yannickooAttached patch uses drupal_get_path and also removes some whitespaces and converts tabs to spaces in the
page.tpl.phpfile.Comment #2
loominade commentedum sorry, it has to be
Comment #3
loominade commentedComment #4
yannickooThen let's do that :)