From: skodak <skodak>
Date: Thu, 4 Jan 2007 12:44:58 +0000 (+0000)
Subject: MDL-7861 various xhtml fixes and fixed layout of add activity/resource menu
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=4f24b3e3864707d945fb7a12e40060ede6c20938;p=moodle.git

MDL-7861 various xhtml fixes and fixed layout of add activity/resource menu
---

diff --git a/blocks/admin_tree/block_admin_tree.php b/blocks/admin_tree/block_admin_tree.php
index af544670b7..e0b6fcdace 100644
--- a/blocks/admin_tree/block_admin_tree.php
+++ b/blocks/admin_tree/block_admin_tree.php
@@ -195,10 +195,10 @@ class block_admin_tree extends block_base {
             $searchcontent = isset($CFG->adminsearchquery) ? $CFG->adminsearchquery : '';
 
             $this->content->footer = '<div class="adminsearchform">'.
-                                     '<form action="'.$CFG->wwwroot.'/admin/search.php" method="get">'.
+                                     '<form action="'.$CFG->wwwroot.'/admin/search.php" method="get"><div>'.
                                      '<label for="query" class="accesshide">'.get_string('searchinsettings', 'admin').'</label>'.
                                      '<input type="text" name="query" id="query" size="8" value="'.s($searchcontent).'" />'.
-                                     '<input type="submit" value="'.get_string('search').'" />'.
+                                     '<input type="submit" value="'.get_string('search').'" /></div>'.
                                      '</form></div>';
         } else {
             $this->content = new stdClass;
diff --git a/blocks/search/block_search.php b/blocks/search/block_search.php
index 96b73ec2d7..f06d6c9d07 100644
--- a/blocks/search/block_search.php
+++ b/blocks/search/block_search.php
@@ -50,11 +50,11 @@
         
         //basic search form
         $this->content->text =
-            '<form name="query" method="get" action="'. $CFG->wwwroot .'/search/query.php">'
+            '<form name="query" method="get" action="'. $CFG->wwwroot .'/search/query.php"><div>'
           . '<label for="block_search_q">'. $label .'</label>'
           . '<input id="block_search_q" type="text" name="query_string" length="50" />'
           . '<input type="submit" value="'.$button.'" />'
-          . '</form>';
+          . '</div></form>';
       } else {
         $this->content->text = "Sorry folks, PHP 5 is needed for the new search module.";
       } //else
diff --git a/blocks/search_forums/block_search_forums.php b/blocks/search_forums/block_search_forums.php
index de679ee13c..0a76909e75 100644
--- a/blocks/search_forums/block_search_forums.php
+++ b/blocks/search_forums/block_search_forums.php
@@ -32,13 +32,13 @@ class block_search_forums extends block_base {
         $button = (isset($CFG->block_search_button)) ? $CFG->block_search_button : get_string('go');
         
         $this->content->text  = '<div class="searchform">';
-        $this->content->text .= '<form name="search" action="'.$CFG->wwwroot.'/mod/forum/search.php" style="display:inline">';
+        $this->content->text .= '<form name="search" action="'.$CFG->wwwroot.'/mod/forum/search.php" style="display:inline"><div>';
         $this->content->text .= '<input name="id" type="hidden" value="'.$this->instance->pageid.'" />';  // course
         $this->content->text .= '<input name="search" type="text" size="16" />';
         $this->content->text .= '<button type="submit" title="'.get_string('search').'">'.$button.'</button><br />'; 
         $this->content->text .= '<a href="'.$CFG->wwwroot.'/mod/forum/search.php?id='.$this->instance->pageid.'">'.$advancedsearch.'</a>';
         $this->content->text .= helpbutton('search', $advancedsearch, 'moodle', true, false, '', true);
-        $this->content->text .= '</form></div>';
+        $this->content->text .= '</div></form></div>';
 
         return $this->content;
     }
diff --git a/course/lib.php b/course/lib.php
index d488074827..9adf446a59 100644
--- a/course/lib.php
+++ b/course/lib.php
@@ -1484,24 +1484,24 @@ function print_section_add_menus($course, $section, $modnames, $vertical=false,
     $straddactivity = get_string('addactivity');
     $straddresource = get_string('addresource');
 
-    $output  = '<div class="section_add_menus" style="text-align: right">';
+    $output  = '<div class="section_add_menus">';
+
+    if (!$vertical) {
+        $output .= '<div class="horizontal">';
+    }
 
     if (!empty($resources)) {
         $output .= popup_form("$CFG->wwwroot/course/mod.php?id=$course->id&amp;section=$section&amp;sesskey=".sesskey()."&amp;add=",
                               $resources, "ressection$section", "", $straddresource, 'resource/types', $straddresource, true);
     }
 
-    if ($vertical) {
-        $output .= '<div>';
-    }
-
     if (!empty($activities)) {
         $output .= ' ';
         $output .= popup_form("$CFG->wwwroot/course/mod.php?id=$course->id&amp;section=$section&amp;sesskey=".sesskey()."&amp;add=",
                     $activities, "section$section", "", $straddactivity, 'mods', $straddactivity, true);
     }
 
-    if ($vertical) {
+    if (!$vertical) {
         $output .= '</div>';
     }
 
diff --git a/lib/weblib.php b/lib/weblib.php
index 378cf3ff80..c42222cce9 100644
--- a/lib/weblib.php
+++ b/lib/weblib.php
@@ -958,13 +958,18 @@ function popup_form($common, $options, $formid, $selected='', $nothing='choose',
 
     // changed reference to document.getElementById('id_abc') instead of document.abc
     // MDL-7861
-    $startoutput = '<form action="'.$CFG->wwwroot.'/course/jumpto.php"'.
+    $output = '<form action="'.$CFG->wwwroot.'/course/jumpto.php"'.
                         ' method="get"'.
                         $target.
                         ' id="'.$formid.'"'.
                         ' class="popupform">';
+    if ($help) {
+        $button = helpbutton($help, $helptext, 'moodle', true, false, '', true);
+    } else {
+        $button = '';
+    }
 
-    $output = '<div><select name="jump" onchange="'.$targetwindow.'.location=document.getElementById(\''.$formid.'\').jump.options[document.getElementById(\''.$formid.'\').jump.selectedIndex].value;">'."\n";
+    $output .= '<div>'.$button.'<select name="jump" onchange="'.$targetwindow.'.location=document.getElementById(\''.$formid.'\').jump.options[document.getElementById(\''.$formid.'\').jump.selectedIndex].value;">'."\n";
 
     if ($nothing != '') {
         $output .= "   <option value=\"javascript:void(0)\">$nothing</option>\n";
@@ -1047,16 +1052,10 @@ function popup_form($common, $options, $formid, $selected='', $nothing='choose',
                "\n//]]>\n".'</script>';
     $output .= '</form>' . "\n";
 
-    if ($help) {
-        $button = helpbutton($help, $helptext, 'moodle', true, false, '', true);
-    } else {
-        $button = '';
-    }
-
     if ($return) {
-        return $startoutput.$button.$output;
+        return $output;
     } else {
-        echo $startoutput.$button.$output;
+        echo $output;
     }
 }
 
@@ -3851,11 +3850,11 @@ function update_course_icon($courseid) {
             $target = ' target="'.$CFG->framename.'"';
         }
 
-        return "<form$target method=\"get\" action=\"$CFG->wwwroot/course/view.php\">".
+        return "<form$target method=\"get\" action=\"$CFG->wwwroot/course/view.php\"><div>".
             "<input type=\"hidden\" name=\"id\" value=\"$courseid\" />".
             "<input type=\"hidden\" name=\"edit\" value=\"$edit\" />".
             "<input type=\"hidden\" name=\"sesskey\" value=\"".sesskey()."\" />".
-            "<input type=\"submit\" value=\"$string\" /></form>";
+            "<input type=\"submit\" value=\"$string\" /></div></form>";
     }
 }
 
diff --git a/theme/standard/styles_layout.css b/theme/standard/styles_layout.css
index 6a794fd1ed..6e4062697b 100644
--- a/theme/standard/styles_layout.css
+++ b/theme/standard/styles_layout.css
@@ -1187,6 +1187,15 @@ span.current {
   list-style: none;
 }
 
+.section_add_menus {
+  text-align:right;
+}
+
+.section_add_menus .horizontal div {
+  display:inline;
+}
+
+
 /*#course-view ul.section li.activity ul li,
 #site-index ul.section li.activity ul li {
   list-style: disc;