Thanks for all the feedback. I will get to work removing the pop-up windows - moving the content into the main page template makes sense and the accessibility advantages make it the obvious choice (easier than going back to HTML strict too!).
I do care about mac users (being a part-time mac user myself). I was aware of an issue in ie5 but not the one in firefox. I'll look into it. The color scheme was an attempt to get away from the traditional view of "organics" - to be a little bolder than the harmonious green palette you may expect. But I know it can look strong! Once again thankyou everyone for taking the time to look at it - it is very valuable feedback. Cheers James -----Original Message----- From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On Behalf Of Peter Firminger Sent: Tuesday, 22 February 2005 11:45 AM To: [email protected] Subject: RE: [WSG] FW: Site review Hi James, As the page http://www.organicexpo.com.au/exhibitors/index.php validates as XHTML 1.0 Strict, this may not mean anything but I have had errors or warnings from validators about white space in html comments in the past. <!--comment--> wasn't acceptable as there needed to be a space either inside the delimiters (--) as in <!-- comment -->. Seems this has been changed in the specs, or at least the validators. I would still suggest doing it that way anyway. This also means that ColdFusion comments <!--- comment ---> should not really be used on pages that are not parsed by ColdFusion (removing them from the source). The links without hrefs in the footer are inaccessible. <a onclick="javascript:newWindow('../articles/terms.php')">terms and conditions</a> Without JavaScript (many smartphones etc. simply don't have it), this won't work and they can't read your terms. A potential legal issue. The code we use (in HTML and only when we really need to) is: <a href="page.htm" target="targetname" onClick="window.open('', 'targetname','toolbar=0,location=0,directories=0,status=0,menubar=0,scro llba rs=auto,resizable=0,width10,height00')" title="Link text (opens in a new window)">Link text</a> This is very stable code that works everywhere. So, either don't use XHTML if you want accessible links to popup windows (due to the lack of the target attribute) or don't use popup windows at all, loading the terms and conditions and privacy policy in the full window. The latter is obviously preferable as people using screen readers without the aid of vision may not know the new window has opened and get completely lost on a page with no navigation aids. Just my thoughts... P ****************************************************** The discussion list for http://webstandardsgroup.org/ See http://webstandardsgroup.org/mail/guidelines.cfm for some hints on posting to the list & getting help ****************************************************** ****************************************************** The discussion list for http://webstandardsgroup.org/ See http://webstandardsgroup.org/mail/guidelines.cfm for some hints on posting to the list & getting help ******************************************************
