Quoting bklaas <[EMAIL PROTECTED]>:


spot on as usual, KDF!

I'm making the necessary changes to all of the pages and will update
svn in a few minutes.

I saw the checkin, and it looks good from viewing that. I can't update until later.

One remaining inconsistency is that I want the playlist controls to
show up on the left, not the right of each table row (left is the
convention throughout the skin, and IMO for good readson). I'm pretty
sure that would have to happen via a custom skin for the plugins, but
it's a minor point compared to getting the nav bar up there...thanks
for the pointer, KDF.

The contentitem object in cmdwrappers should allow you to set the controls wherever you like. It all depends on where in the [% BLOCK contentitem -%] that you PROCESS the controls. Check Moser and Default skin for examples. You can just move it to before the [% content %] section. To make it work originally, I just pasted from Default so that the plugins woudln't be completely wrecked. There are some minor errors that have to be cleaned up. What you probably want is more like this:

[% BLOCK contentitem -%]
<tr><td class="listing" align="left">
        [% IF controls %]
        <div class="browsedbControls">
        [%- PROCESS $controls %][% ELSE %]<td></td><td></td>[% END %]
        </div></td>
<td class="listing" align="left">
        [%- IF anchor %]
                <a name="[% anchor %]"></a>
        [%- END %]

        <div class="[% lineclass || 'browsedbListItem' %]">

        [%- content | indent(4) %]
        </td>
</tr>
[%- END %]

It isn't perfect, but it will do until a more complete move from tables to css is done.

-kdf




-kdf
_______________________________________________
Discuss mailing list
[email protected]
http://lists.slimdevices.com/lists/listinfo/discuss

Reply via email to