Monni wrote on Oct 20^th, 2014 at 8:42pm:


I think this is one of those 'choose your battles'. Plain numeric ID attributes are okay in HTML5 and work (even if they're not valid) in XHTML and HTML4. (Personally, I think we should get rid of the W3C validation links except for admins testing templates because the moment you do anything browser specific your css becomes invalid.  And we have a LOT of invalid css that works just fine.)

Index: cgi-bin/yabb2/Languages/English/BoardIndex.lng
===================================================================
--- cgi-bin/yabb2/Languages/English/BoardIndex.lng	(revision 1600)
+++ cgi-bin/yabb2/Languages/English/BoardIndex.lng	(working copy)
@@ -62,6 +62,7 @@
 '524' => 'In',
 '525' => 'By',
 '526' => 'Redirected Hits',
+'643' => 'Up',
 '685' => 'Info Center',
 '791' => 'View the',
 '792' => 'most recent posts',
@@ -107,4 +108,4 @@
 '6' => 'This category contains no new posts.',
 );

-1;
\ No newline at end of file
+1;
Index: cgi-bin/yabb2/Sources/BoardIndex.pm
===================================================================
--- cgi-bin/yabb2/Sources/BoardIndex.pm	(revision 1600)
+++ cgi-bin/yabb2/Sources/BoardIndex.pm	(working copy)
@@ -1374,12 +1374,12 @@
     $template_catnames =~ s/,\Z//xsm;
     $template_boardnames =~ s/,\Z//xsm;
     $yymain .= qq~
-<script type="text/javascript">
+<script type="text/javascript">//<![CDATA[
     var catNames = [$template_catnames];
     var boardNames = [$template_boardnames];
     var boardOpen = "";
     var subboardOpen = "";
-    var arrowup = '<img src="$imagesdir/$brd_arrowup" class="brd_arrow" />';
+    var arrowup = '<img src="$imagesdir/$brd_arrowup" class="brd_arrow" alt="$boardindex_txt{'643'}" />';
     var openbutton = "$imagesdir/$brd_dropdown";
     var closebutton = "$imagesdir/$brd_dropup";
     var opensubbutton = "$imagesdir/$sub_arrow_dn";
@@ -1397,7 +1397,7 @@
     var brd_img_idw = $brd_img_idw;
     var brd_img_idh = $brd_img_idh;
     var fix_brd_size = $fix_brd_img_size;
-</script>~;
+//]]></script>~;

     # don't show info center, login, etc. if we're calling from sub boards
     if ( !$subboard_sel ) {
@@ -1784,12 +1784,12 @@
         elsif ($subboard_sel) {
             if ($brd_count) {
                 $boardindex_template = qq~
-                        <script type="text/javascript">
+                        <script type="text/javascript">//<![CDATA[
                         var catNames = [$template_catnames];
                         var boardNames = [$template_boardnames];
                         var boardOpen = "";
                         var subboardOpen = "";
-                        var arrowup = '<img src="$imagesdir/$brd_arrowup" class="brd_arrow" />';
+                        var arrowup = '<img src="$imagesdir/$brd_arrowup" class="brd_arrow" alt="$boardindex_txt{'643'}" />';
                         var openbutton = "$imagesdir/$brd_dropdown";
                         var closebutton = "$imagesdir/$brd_dropup";
                         var loadimg = "$imagesdir/$brd_loadbar";
@@ -1799,7 +1799,7 @@
                         var insertindex;
                         var insertcat;
                         var prev_subcount;
-                        </script>
+                        //]]></script>
                         $boardindex_template
 ~;
             }
@@ -2025,4 +2025,4 @@
     }
 }

-1;
\ No newline at end of file
+1;
Index: cgi-bin/yabb2/Sources/Subs.pm
===================================================================
--- cgi-bin/yabb2/Sources/Subs.pm	(revision 1600)
+++ cgi-bin/yabb2/Sources/Subs.pm	(working copy)
@@ -715,7 +715,7 @@
     # in a very strict way. (error 406)
     # Take the comments out of the following two lines if you had this problem.
     # $output =~ s/($scripturl\?)([^'"]+)/ $1 . URL_modify($2) /eg;
-    # sub URL_modify { my $x = shift; $x =~ s/;/&/g; $x; }
+    # sub URL_modify { my $x = shift; $x =~ s/;/&amp;/g; $x; }
     # End of workaround

     if ( !$copyright ) {
Index: cgi-bin/yabb2/Templates/default/Micon.def
===================================================================
--- cgi-bin/yabb2/Templates/default/Micon.def	(revision 1600)
+++ cgi-bin/yabb2/Templates/default/Micon.def	(working copy)
@@ -252,7 +252,7 @@
 );

 $newload = qq~
-        <script type="text/javascript">
+        <script type="text/javascript">//<![CDATA[
         var new_mess = "$newload{'new_mess'}";
         var brd_new = "$newload{'brd_new'}";
         var brd_old = "$newload{'brd_old'}";
@@ -264,7 +264,7 @@
         var imopen2 = "$newload{'imopen2'}";
         var imclose = "$newload{'imclose'}";
         var imclose2 = "$newload{'imclose2'}";
-        </script>~;
+        //]]></script>~;

 #####################################################################
 #                   JS Variables for IMPost, Post                   #
@@ -315,4 +315,4 @@
 if ( $MenuType == 1 || $UseMenuT == 1) {
     $my_sep = q{ | };
 }
-1;
\ No newline at end of file
+1;
 

This drops the error count on board index to 1, which is obviously false positive.