Here is a bunch more patches for XHTML compliance. These patches mostly deal with the genre portion of the web interface. I will try to go through all of the web interface and bring it up to XHTML compliance. I am validating them against http://validator.w3.org/

Ryan
Index: src/www/htdocs/genre.rpy
===================================================================
--- src/www/htdocs/genre.rpy    (revision 8811)
+++ src/www/htdocs/genre.rpy    (working copy)
@@ -43,15 +43,15 @@
 # need sub to get list of possible categories

     def makecategorybox(self, categories, category):
-        retval = u'<select name="category">\n'
+        retval = u'<select name="category">'
         category = Unicode( category )
         for cat in categories:
             cat = Unicode( cat )
             retval += u'<option value="%s" ' % cat
             if cat == category:
-                retval += u'SELECTED '
-            retval += u'>%s</option>\n' % cat
-        retval += u'</select>\n'
+               retval += u'selected="selected"'
+            retval += u'>%s</option>' % cat
+        retval += u'</select>'
         return retval

     def _render(self, request):
@@ -101,19 +101,19 @@

         stime=''
         if mfrguideinput:
-            stime='<input name="stime" type="hidden" value="%i">' % 
int(mfrguideinput)
+            stime='<input name="stime" type="hidden" value="%i"/>' % 
int(mfrguideinput)
         keepcat = ''
         if category:
             keepcat = '&category=%s' % category
         bforcell=''
         acelltime=mfrnextguide + 60
-        aftercell='&nbsp;&nbsp;&nbsp;<a href="genre.rpy?stime=%i%s"><img 
src="images/RightArrow.png" border="0"></a>' % (acelltime, keepcat)
+        aftercell='&nbsp;&nbsp;&nbsp;<a href="genre.rpy?stime=%i%s"><img 
src="images/RightArrow.png" alt="-->" style="border-style: 
none"></img></a></p>' % (acelltime, keepcat)
         if mfrprevguide > 0:
-            bforcell='<a href="genre.rpy?stime=%i%s"><img 
src="images/LeftArrow.png" border="0"></a>&nbsp;&nbsp;&nbsp;' % (mfrprevguide, 
keepcat)
+            bforcell='<a href="genre.rpy?stime=%i%s"><img 
src="images/LeftArrow.png" alt="<--" style="border-style: 
none"></a>&nbsp;&nbsp;&nbsp;' % (mfrprevguide, keepcat)

         fv.tableOpen('border="0" cellpadding="4" cellspacing="1" width="100%"')
         fv.tableRowOpen('class="chanrow"')
-        
fv.tableCell('<form>'+bforcell+_('Show')+'&nbsp;'+_('Category')+':&nbsp;' + 
self.makecategorybox(allcategories, category)+stime+'<input type=submit 
value="'+_('Change')+'">'+aftercell+'</form>', 'class="guidehead"')
+        fv.tableCell('<form 
action=""><p>'+bforcell+('Show')+'&nbsp;'+_('Category')+':&nbsp;&nbsp;'+self.makecategorybox(allcategories,
 category)+stime+'&nbsp;<input type="submit" 
value="'+_('Change')+'"></input>'+aftercell+'</form>', 'class="guidehead"')
         fv.tableRowClose()
         fv.tableClose()

@@ -197,8 +197,8 @@
             u"            <b>"+_('Start')+u":</b> <span 
id=\"program-start\"></span>, \n"\
             u"            <b>"+_('Stop')+u":</b> <span 
id=\"program-end\"></span>, \n"\
             u"            <b>"+_('Runtime')+u":</b> <span 
id=\"program-runtime\"></span> min\n"\
-            u"            </td>\n"\
             u"         </td>\n"\
+            u"         </tr>\n"\
             u"      </tbody>\n"\
             u"      <tfoot>\n"\
             u"         <tr>\n"\
Index: src/www/web_types.py
===================================================================
--- src/www/web_types.py        (revision 8811)
+++ src/www/web_types.py        (working copy)
@@ -104,13 +100,13 @@
         if style != None:
             self.res += '\t<link rel="stylesheet" href="styles/main.css" 
type="text/css" />\n'
         if script != None:
-            self.res += '\t<script language="JavaScript" 
src="'+script+'"></script>\n'
+            self.res += '\t<script type="text/javascript" 
src="'+script+'"></script>\n'
         self.res += '</head>\n'
         self.res += '\n\n\n\n<body>\n'
         # Header
         self.res += '<!-- Header Logo and Status Line -->\n'
         self.res += '<div id="titlebar"><span class="name">'\
-            +'<a href="http://freevo.sourceforge.net/"; 
target="_blank">Freevo</a></span></div>\n'
+            +'<a 
href="http://freevo.sourceforge.net/";>Freevo</a></span></div>\n'

         items = [(_('Home'),_('Home'),'%sindex.rpy' % str(strprefix)),
                  (_('TV Guide'),_('View TV Listings'),'%sguide.rpy' % 
str(strprefix)),
@@ -254,7 +250,7 @@
         return breadcrumb

     def printPassword(self, password):
-        self.res += """<script language="JavaScript"> <!--
+        self.res += """<script type="text/javascript"> <!--

         var password;

@@ -270,7 +266,7 @@
         </script>"""

     def printImagePopup(self):
-        self.res += """<script language="JavaScript" type="text/javascript" 
style="display:none;">
+        self.res += """<script type="text/javascript" style="display:none;">
         function openfoto(loc,width,height){
             var 
params="toolbar=no,location=no,status=no,menubar=no,resizable=no,scrollbars=no,top=0,left=0,width="+width+",height="+height;
             foto = window.open("fileinfo.rpy?img="+loc,"Images",params);
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Freevo-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/freevo-devel

Reply via email to