MDL-7861, fixing xhtml strict bugs

diff --git a/mod/data/field.php b/mod/data/field.php
index a4669b41b7fe7175f505a870685753d8c927a1fd..f083022905912eb2084efdff34247fba82a97f84 100755 (executable)
--- a/mod/data/field.php
+++ b/mod/data/field.php
@@ -267,21 +267,18 @@
 
                     $table->data[] = array(
 
-
-
                     '<a href="field.php?mode=display&amp;d='.$data->id.
                     '&amp;fid='.$field->field->id.'&amp;sesskey='.sesskey().'">'.$field->field->name.'</a>',
 
-
                     $field->image().'&nbsp;'.get_string($field->type, 'data'),
 
                     shorten_text($field->field->description, 30),
 
                     '<a href="field.php?d='.$data->id.'&amp;mode=display&amp;fid='.$field->field->id.'&amp;sesskey='.sesskey().'">'.
-                    '<img src="'.$CFG->pixpath.'/t/edit.gif" height="11" width="11" border="0" alt="'.get_string('edit').'" title="'.get_string('edit').'" /></a>'.
+                    '<img src="'.$CFG->pixpath.'/t/edit.gif" height="11" width="11" alt="'.get_string('edit').'" title="'.get_string('edit').'" /></a>'.
                     '&nbsp;'.
                     '<a href="field.php?d='.$data->id.'&amp;mode=delete&amp;fid='.$field->field->id.'&amp;sesskey='.sesskey().'">'.
-                    '<img src="'.$CFG->pixpath.'/t/delete.gif" height="11" width="11" border="0" alt="'.get_string('delete').'" title="'.get_string('delete').'" /></a>'
+                    '<img src="'.$CFG->pixpath.'/t/delete.gif" height="11" width="11" alt="'.get_string('delete').'" title="'.get_string('delete').'" /></a>'
 
                     );
                 }
@@ -290,7 +287,7 @@
         }
 
 
-        echo '<div class="fieldadd" align="center">';
+        echo '<div class="fieldadd">';
         echo get_string('newfield','data').': ';
         popup_form($CFG->wwwroot.'/mod/data/field.php?d='.$data->id.'&amp;mode=new&amp;sesskey='.
                 sesskey().'&amp;newtype=', $menufield, 'fieldform', '', 'choose');
@@ -298,8 +295,9 @@
         echo '</div>';
 
         if ($fields = get_records('data_fields','dataid',$data->id)) {
-            echo '<div class="sortdefault" align="center">';
+            echo '<div class="sortdefault">';
             echo '<form id="sortdefault" action="'.$CFG->wwwroot.'/mod/data/field.php" method="get">';
+            echo '<fieldset class="invisiblefieldset">';
             echo '<input type="hidden" name="d" value="'.$data->id.'" />';
             echo '<input type="hidden" name="mode" value="sort" />';
             echo '<input type="hidden" name="sesskey" value="'.sesskey().'" />';
@@ -319,7 +317,7 @@
                              1 => get_string('descending', 'data'));
             choose_from_menu($options, 'defaultsortdir', $data->defaultsortdir, '');
             echo '<input type="submit" value="'.get_string('go').'" />';
-
+            echo '</fieldset>';
             echo '</form>';
             echo '</div>';
         }

diff --git a/mod/data/lib.php b/mod/data/lib.php
index 3ca39f9792db6faae955265d564376dd0ac590a3..3e0052b7a8e589c042e751471a0039b25982c472 100755 (executable)
--- a/mod/data/lib.php
+++ b/mod/data/lib.php
@@ -303,7 +303,7 @@ class data_field_base {     /// Base class for Database Field Types (see field/*
 
         $str = '<a href="field.php?d='.$this->data->id.'&amp;fid='.$this->field->id.'&amp;mode=display&amp;sesskey='.sesskey().'">';
         $str .= '<img src="'.$CFG->modpixpath.'/data/field/'.$this->type.'/icon.gif" ';
-        $str .= 'height="'.$this->iconheight.'" width="'.$this->iconwidth.'" border="0" alt="'.$this->type.'" title="'.$this->type.'" /></a>';
+        $str .= 'height="'.$this->iconheight.'" width="'.$this->iconwidth.'" alt="'.$this->type.'" title="'.$this->type.'" /></a>';
         return $str;
     }
 

diff --git a/theme/standard/styles_layout.css b/theme/standard/styles_layout.css
index 8f803641f99b740c1967c675c3b736bed85fafbc..cd0a90faf032f8948fcb87d91afca65094d7af22 100644 (file)
--- a/theme/standard/styles_layout.css
+++ b/theme/standard/styles_layout.css
@@ -2023,6 +2023,14 @@ body#user-index .rolesform {
  *** Modules: Choice
  ***/
 
+/***
+ *** Modules: Data
+ ***/
+.fieldadd,
+.sortdefault {
+  text-align:center; 
+}
+
 /***
  *** Modules: Forum
  ***/