By GPirie on

I've just finished creating the Yorkshire TA website, and would like to hear your feedback on both design and functionality. It's a little quiet as it only went live this morning but this should increase over the next week or so.

Thanks

www.yorkshireta.co.uk

⋅ Categories: Drupal 6.x

Comments

matt_harrold’s picture

matt_harrold commented

Nicely done ... I think the look of your site is clean and easy on the eye.

How about Clean URL support? perhaps go to admin/settings/clean-urls and activate them, they'll make the site URLs look prettier.

Anonymous’s picture

Anonymous (not verified) commented

yes, and more search-engine friendly as well! nice site.

jstarek’s picture

jstarek commented

Using Konqueror 3.5.8, I get leftover code in your footer:

pageTracker._trackPageview();

Perhaps you'd like to look into this. Otherwise, a nice and clearly laid out site!

GPirie’s picture

GPirie commented

Aye I noticed it that as well, currently trying to sort it out although I'm struggling a bit. Anyone have any suggestions?

www.yorkshireta.co.uk

baronmunchowsen’s picture

baronmunchowsen commented

Your code:

<script type="text/javascript">document.write(unescape("%3Cscript src='http://www.google-analytics.com/ga.js' type='text/javascript'%3E%3C/script%3E"));</script>
<script type="text/javascript">var pageTracker = _gat._getTracker("gpirie10");pageTracker._initData();<script type="text/javascript">
var gaJsHost = (("https:" == document.location.protocol) ? "https://ssl." : "http://www.");
document.write(unescape("%3Cscript src='" + gaJsHost + "google-analytics.com/ga.js' type='text/javascript'%3E%3C/script%3E"));
</script>
<script type="text/javascript">
var pageTracker = _gat._getTracker("UA-3642084-6");
pageTracker._initData();
pageTracker._trackPageview();

</script>pageTracker._trackPageview();</script>
<script type="text/javascript" src="/modules/google_analytics/downloadtracker.js?o"></script>

I think you're missing an opening

tag. 
...
pageTracker._initData();
pageTracker._trackPageview();

</script>pageTracker._trackPageview();</script>
...
should become: 
...
pageTracker._initData();
pageTracker._trackPageview();

</script>
<script>pageTracker._trackPageview();</script>
...
But I might be wrong.