cvs: pear /HTML_QuickForm_advmultiselect/examples qfams_custom_4.php

[email protected] ("Laurent Laville")
Newsgroups php.pear.cvs
Message-ID <cvsfarell1209213944@cvsserver>
farell		Sat Apr 26 12:45:44 2008 UTC

  Modified files:              
    /pear/HTML_QuickForm_advmultiselect/examples	qfams_custom_4.php 
  Log:
  related to bug 13680, make example more XHTML compliant
  
http://cvs.php.net/viewvc.cgi/pear/HTML_QuickForm_advmultiselect/examples/qfams_custom_4.php?r1=1.5&r2=1.6&diff_format=u
Index: pear/HTML_QuickForm_advmultiselect/examples/qfams_custom_4.php
diff -u pear/HTML_QuickForm_advmultiselect/examples/qfams_custom_4.php:1.5 pear/HTML_QuickForm_advmultiselect/examples/qfams_custom_4.php:1.6
--- pear/HTML_QuickForm_advmultiselect/examples/qfams_custom_4.php:1.5	Fri Jan  5 15:35:33 2007
+++ pear/HTML_QuickForm_advmultiselect/examples/qfams_custom_4.php	Sat Apr 26 12:45:44 2008
@@ -4,7 +4,7 @@
  * that present alternatively a dual multsi-select
  * or a single checkboxes with fancy attributes.
  *
- * @version    $Id: qfams_custom_4.php,v 1.5 2007/01/05 15:35:33 farell Exp $
+ * @version    $Id: qfams_custom_4.php,v 1.6 2008/04/26 12:45:44 farell Exp $
  * @author     Laurent Laville <[email protected]>
  * @package    HTML_QuickForm_advmultiselect
  * @subpackage Examples
@@ -109,7 +109,7 @@
 $buttons[] =& $form->createElement('reset', null, 'Reset');
 $buttons[] =& $form->createElement('checkbox', 'multiselect', null,
                                    'use dual select boxes layout');
-$form->addGroup($buttons);
+$form->addGroup($buttons, null, '&nbsp;');
 
 $form->addRule('name', 'Your name is required', 'required');
 $form->addGroupRule('fruit', 'At least one fruit is required', 'required', null, 1);
@@ -187,7 +187,7 @@
 
     printf("<p>Welcome <b>%s</b> you've selected these fruits:</p>",
            $clean['name']);
-    echo implode(', ', $clean['fruit']);
+    echo '<p>' . implode(', ', $clean['fruit']) . '</p>';
 }
 $form->display();
 ?>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.