Add Logos to Sponsor Details (PATCH)

[spatialguru]

Following from a discussion here: http://pkp.sfu.ca/support/forum/viewtopic.php?f=3&t=2368&p=11652#p11652 I've dug into the code and generated a very simple patch for 2.1. Hopefully this isn't redundant with new features in the CVS, but it sure helped me understand simple parts of the code. Think this will port back to 2.0 easily?

What this does

• Adds Logo Image element to the schedConf Step 1 details forms for each Sponsor. A URL to an image.
• Displays the image beneath the sponsor name on the About - Organising Team page.

The patches..
It's really only done by adding 7 simple lines, mostly cloned and modified from other existing lines! Don't be afraid

templates/manager/schedConfSetup/step1.tpl

Code: Select all

--- templates/manager/schedConfSetup/step1.tpl.orig     2008-06-10 13:35:01.000000000 -0400
+++ templates/manager/schedConfSetup/step1.tpl  2008-06-10 14:52:14.000000000 -0400
@@ -178,6 +178,10 @@
                <td width="20%" class="label">{fieldLabel name="sponsors-$sponsorId-url" key="common.url"}</td>
                <td width="80%" class="value"><input type="text" name="sponsors[{$sponsorId|escape}][url]" id="sponsors-{$sponsorId|escape}-url" value="{$sponsor.url|escape}" size="40" maxlength="255" class="textField" /></td>
        </tr>
+        <tr valign="top">
+                <td width="20%" class="label">{fieldLabel name="sponsors-$sponsorId-logourl" key="manager.setup.layout.useImageLogo"}</td>
+                <td width="80%" class="value"><input type="text" name="sponsors[{$sponsorId|escape}][logourl]" id="sponsors-{$sponsorId|escape}-logo" value="{$sponsor.logourl|escape}" size="40" maxlength="255" class="textField" /></td>
+        </tr>
        {if !$smarty.foreach.sponsors.last}
        <tr valign="top">
                <td colspan="2" class="separator">&nbsp;</td>

• A couple questions here.. could I use a better key for the label? Ideally it'd also say "URL" but I wasn't sure how to concatenate two items in a key. I also didn't want to add anything to locale files.
• Is logourl an okay variable name to use? I kept it as simple as a I could and hoped to not overlap usage in other places.

templates/about/conferenceSponsorship.tpl

Code: Select all

--- templates/about/conferenceSponsorship.tpl.orig      2008-06-10 14:53:23.000000000 -0400
+++ templates/about/conferenceSponsorship.tpl   2008-06-10 15:04:51.000000000 -0400
@@ -34,6 +34,9 @@
                {else}
                        <li>{$sponsor.institution|escape}</li>
                {/if}
+               {if $sponsor.logourl}
+                       <li style="list-style-type:none"><a href="{$sponsor.url|escape}"><img src="{$sponsor.logourl|escape}"></a></li>
+               {/if}
        {/if}
        {/foreach}
 </ul>

Note in the last snippit, I inserted the logo as a list item, but set the style to none, so no bullet would be shown. I'm not sure if I should hardcode this or refer to an existing style class. Any suggestions?

What I want to do next

• Write instructions for Creating a new block
• Publish Sponsor Logos Block Plugin -- now working

[spatialguru]

With the stylesheet I was using on our site, I got some ugly graphic borders when viewing in Firefox. So I recommend changing this line in the template:

Code: Select all

<img src="{$sponsor.logourl|escape}">
TO
<img src="{$sponsor.logourl|escape}" border="0">


[spatialguru]

As I started to create the sponsor logo plugin block, I realised I needed a couple more fields for finer control. So I've now added two more fields as well. One for having a smaller or block-appropriate logo, the other for specifying an order/ranking. For example, now you could have a sponsor on the Organising Team page, with a large logo, and a different logo in the block. And you could put your biggest sponsors at the top of the block.

I haven't figured out how to reorder them on the organising team section though. Any hints? Patch file attached, but still testing.

Always open to naming convention, patch generation and UI improvement or standard suggestions. I'm very new at this