update to Whitespace theme

[Andy C]

With yet more assistance from Michael Harris, I have fixed a couple of
irritating bugs in the Whitespace theme.

- All pages should now validate as XHTML.
- Malformed RSS feed in header fixed
- Search functionality implemented.
- Height of footer reduced to fit.
- Title is now '<post>/<tag> - <blog>' for SEO.

The couple of people using this theme might want to grab the latest
version from

http://www.nbrightside.com/blog/user/files/themes/whitespace/whitespace-0.2.tar.gz

Peace, love, empathy
--
Andy

[shep]

I found another issue. the right sidebar isn't listed in the right
sidebar div, so i changed that so it would be in the div and the
search form wouldn't be so close to the text

> http://www.nbrightside.com/blog/user/files/themes/whitespace/whitespa...

[Andy C]

Hi Mike

There was something about that search bar that was nagging away at me
like a dripping tap.

http://www.nbrightside.com/blog/user/files/themes/whitespace/whitespace-0.3.tar.gz

Thanks a lot
--
Andy

[sfong15]

Hi Andy,

I've downloaded your 0.3 but still seeing it as 0.2 in the xml file
and dashboard. Will you be sharing your latest version? I really
like your left sidebar with 3 tabs!

Thanks.

On Mar 11, 4:29 pm, Andy C <andyc...@gmail.com> wrote:
> Hi Mike
>
> There was something about that search bar that was nagging away at me
> like a dripping tap.
>

> http://www.nbrightside.com/blog/user/files/themes/whitespace/whitespa...

>
> Thanks a lot
> --
> Andy
>
> On Mar 11, 2:30 am, shep <mikeschep...@gmail.com> wrote:
>
> > I found another issue.  the right sidebar isn't listed in the right
> > sidebar div, so i changed that so it would be in the div and the
> > search form wouldn't be so close to the text
>
> > On Mar 5, 4:24 pm, Andy C <andyc...@gmail.com> wrote:
>
> > > With yet more assistance from Michael Harris, I have fixed a couple of
> > > irritating bugs in theWhitespacetheme.
>
> > >     - All pages should now validate as XHTML.
> > >     - Malformed RSS feed in header fixed
> > >     - Search functionality implemented.
> > >     - Height of footer reduced to fit.
> > >     - Title is now '<post>/<tag> - <blog>' for SEO.
>

> > > The couple of people using thisthememight want to grab the latest

[Andy C]

Hi

You actually have whitespace V0.3 but I foolishly omitted to update
the version number in 'theme.xml'. Doh !

If you care, simply edit the file and update the version yourself.

I was going to release 0.4 which would use Rick Cockburn's metatitle
plugin which simplifies the theme but I encountered some problems that
I have asked Rick about.

Have fun !
--
Andy

[rick c]

//snip

On Apr 15, 2:06 pm, Andy C <andyc...@gmail.com> wrote:
> I was going to release 0.4 which would use Rick Cockburn's metatitle
> plugin which simplifies the theme but I encountered some problems that
> I have asked Rick about.

Andy,

It's Cockrum.

I haven't gotten your message about problems with the MetaTitle
plugin. What's up?

Rick
http://sagrising.cockrumpublishing.com

[Andy C]

Hi Rick

Whoops - apologies for getting your surname wrong.

I just wondered whether this plugin works on the latest SVN. Using SVN
r1554 and K2, the title isn't always correct.

For example, when on category 'blogging', the title displays as
'archive - <blog>'

http://www.nbrightside.com/sandpit/tag/blogging

Most of the other pages seem OK but any tag seems to display as
'archive'.

Maybe it's something weird with my data/pages
--
Andy

[rick c]

Andy,

No. Your data and pages are okay. I screwed up. I used a function to
get the tag text that I have in Arctic Ice's theme file, not standard
Habari api. I'll get it fixed and an update posted tonight.

Rick

[Andy C]

No problem. Just pleased to help out. Look forward to the patch.

[rick c]

Andy,

It should be fixed now. Go ahead and download at
http://sagrising.cockrumpublishing.com/files/metatitle.zip. Let me
know how it goes.

Rick

[Andy C]

Hi Rick

Yup. Works fine.

One question - initially, I thought I would be able to remove this
code from 'header.php'

<title><?php
if( ( $request->display_entry || $request->display_page ) &&
isset( $post ) ) {
echo "{$post->title} - ";
} elseif( $request->display_entries_by_tag &&
isset( $posts ) ) {
echo $theme-
>get_tag_text(Controller::get_var('tag')) . ' - ';
}
?>
<?php Options::out( 'title' ) ?>
</title>

....and all the work would be done by the single call to
<?php $theme->header(); ?>

However, this doesn't appear to be the case. If I remove the original
<title> blog, I get blank titles for some pages but the plugin does
create a title for a tag page ('Tag Archive - Blog')

I also note your Arctic Ice theme also includes a similar <title>
blog.

Is my understanding correct ?

On Apr 15, 9:14 pm, rick c <rickcock...@gmail.com> wrote:
> Andy,
>

> It should be fixed now. Go ahead and download athttp://sagrising.cockrumpublishing.com/files/metatitle.zip. Let me

[rick c]

Andy,

I'm glad it works.

Yes, right now you need to leave a title tag in the header. It can be
empty ( as with <title></title> ), but it has to be there. At this
point, I'm happy as punch just to get it working. :) Later I'll add
the case for someone who doesn't have a title tag in their theme at
all. I haven't bothered removing the contents of the title tag in
Arctic Ice because the plugin is new and i don't want to require the
use of the plugin with the theme.

Rick
http://sagrising.cockrumpublishing.com

[Andy C]

Hi Rick

Understood. Thanks.

PS. Also corrected your surname in my recent blog entry.

Andy

On Apr 15, 11:07 pm, rick c <rickcock...@gmail.com> wrote:
> Andy,
>

> I'm glad it works.
>
> Yes, right now you need to leave a title tag in the header. It can be
> empty ( as with <title></title> ), but it has to be there. At this
> point, I'm happy as punch just to get it working. :) Later I'll add
> the case for someone who doesn't have a title tag in their theme at
> all. I haven't bothered removing the contents of the title tag in
> Arctic Ice because the plugin is new and i don't want to require the
> use of the plugin with the theme.
>

> Rickhttp://sagrising.cockrumpublishing.com