You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by ad...@apache.org on 2013/09/23 16:32:32 UTC

svn commit: r1525605 - /ofbiz/trunk/framework/widget/templates/htmlMenuMacroLibrary.ftl

Author: adrianc
Date: Mon Sep 23 14:32:32 2013
New Revision: 1525605

URL: http://svn.apache.org/r1525605
Log:
Fixed a bug in the menu widget rendering.

https://issues.apache.org/jira/browse/OFBIZ-5326

Modified:
    ofbiz/trunk/framework/widget/templates/htmlMenuMacroLibrary.ftl

Modified: ofbiz/trunk/framework/widget/templates/htmlMenuMacroLibrary.ftl
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/widget/templates/htmlMenuMacroLibrary.ftl?rev=1525605&r1=1525604&r2=1525605&view=diff
==============================================================================
--- ofbiz/trunk/framework/widget/templates/htmlMenuMacroLibrary.ftl (original)
+++ ofbiz/trunk/framework/widget/templates/htmlMenuMacroLibrary.ftl Mon Sep 23 14:32:32 2013
@@ -17,10 +17,11 @@ specific language governing permissions 
 under the License.
 -->
 
-<#macro renderMenuBegin boundaryComment id style title>
+<#macro renderMenuBegin boundaryComment="" id="" style="" title="">
   <#if boundaryComment?has_content>
 <!-- ${boundaryComment} -->
   </#if>
+  <#-- FIXME: A menu could have an ID and a style, but some visual themes break if both are used. -->
   <div<#if id?has_content> id="${id}"<#elseif style?has_content> class="${style}"</#if>>
   <#if title?has_content>
   <h2>${title}</h2>
@@ -30,7 +31,7 @@ under the License.
       <ul>
 </#macro>
 
-<#macro renderMenuEnd boundaryComment>
+<#macro renderMenuEnd boundaryComment="">
       </ul>
     </li>
   </ul>