You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@click.apache.org by sa...@apache.org on 2010/06/05 07:33:29 UTC

svn commit: r951654 - in /click/trunk/click: documentation/docs/roadmap-changes.html extras/src/org/apache/click/extras/control/Menu.java

Author: sabob
Date: Sat Jun  5 05:33:29 2010
New Revision: 951654

URL: http://svn.apache.org/viewvc?rev=951654&view=rev
Log:
fixed menu not rendering separator. CLK-681

Modified:
    click/trunk/click/documentation/docs/roadmap-changes.html
    click/trunk/click/extras/src/org/apache/click/extras/control/Menu.java

Modified: click/trunk/click/documentation/docs/roadmap-changes.html
URL: http://svn.apache.org/viewvc/click/trunk/click/documentation/docs/roadmap-changes.html?rev=951654&r1=951653&r2=951654&view=diff
==============================================================================
--- click/trunk/click/documentation/docs/roadmap-changes.html (original)
+++ click/trunk/click/documentation/docs/roadmap-changes.html Sat Jun  5 05:33:29 2010
@@ -84,9 +84,13 @@ includes improved Ajax support and @Bind
       </div>
       <ul style="padding: 0em; margin-left:0em;margin-bottom: 2em">
           <li class="change">
+              Fixed menu not rendering separator
+              [<a target="_blank" href="https://issues.apache.org/jira/browse/CLK-681">CLK-681</a>].
+          </li>
+          <li class="change">
               Update to the latest version for some of the JARs dependencies: log4j-1.2.16, freemarker-2.3.16,
               commons-codec-1.4, and commons-lang-2.5.
-          </li>          
+          </li>
       </ul>
   </dd>
   <dt><a name="2.2.0"></a>

Modified: click/trunk/click/extras/src/org/apache/click/extras/control/Menu.java
URL: http://svn.apache.org/viewvc/click/trunk/click/extras/src/org/apache/click/extras/control/Menu.java?rev=951654&r1=951653&r2=951654&view=diff
==============================================================================
--- click/trunk/click/extras/src/org/apache/click/extras/control/Menu.java (original)
+++ click/trunk/click/extras/src/org/apache/click/extras/control/Menu.java Sat Jun  5 05:33:29 2010
@@ -1259,8 +1259,8 @@ public class Menu extends AbstractContro
                 renderMenuListItemAttributes(buffer, child, depth);
                 buffer.closeTag();
 
-                if (menu.isSeparator()) {
-                    renderSeparator(buffer, menu);
+                if (child.isSeparator()) {
+                    renderSeparator(buffer, child);
                 } else {
                     renderMenuLink(buffer, child);
                 }