svn commit: r816856 - in /ofbiz/trunk/themes/bizznesstime/includes: appbar.ftl header.ftl secondary-appbar.ftl

[lektran]

Author: lektran
Date: Sat Sep 19 05:47:19 2009
New Revision: 816856

URL: http://svn.apache.org/viewvc?rev=816856&view=rev
Log:
A few xhtml validation fixes, there's still one I don't know how to fix: a div isn't allowed inside a span

Modified:
    ofbiz/trunk/themes/bizznesstime/includes/appbar.ftl
    ofbiz/trunk/themes/bizznesstime/includes/header.ftl
    ofbiz/trunk/themes/bizznesstime/includes/secondary-appbar.ftl

Modified: ofbiz/trunk/themes/bizznesstime/includes/appbar.ftl
URL: http://svn.apache.org/viewvc/ofbiz/trunk/themes/bizznesstime/includes/appbar.ftl?rev=816856&r1=816855&r2=816856&view=diff
==============================================================================
--- ofbiz/trunk/themes/bizznesstime/includes/appbar.ftl (original)
+++ ofbiz/trunk/themes/bizznesstime/includes/appbar.ftl Sat Sep 19 05:47:19 2009
@@ -27,7 +27,7 @@
             <h2 class="contracted">${uiLabelMap.CommonApplications}</h2>
             <div id="header-nav" class="clearfix" style="display:none">
                 <ul>
-                <h4>${uiLabelMap.CommonPrimaryApps}</h4>
+                <li><h4>${uiLabelMap.CommonPrimaryApps}</h4></li>
                     <#list displayApps as display>
               <#assign thisApp = display.getContextRoot()>
               <#assign permission = true>

Modified: ofbiz/trunk/themes/bizznesstime/includes/header.ftl
URL: http://svn.apache.org/viewvc/ofbiz/trunk/themes/bizznesstime/includes/header.ftl?rev=816856&r1=816855&r2=816856&view=diff
==============================================================================
--- ofbiz/trunk/themes/bizznesstime/includes/header.ftl (original)
+++ ofbiz/trunk/themes/bizznesstime/includes/header.ftl Sat Sep 19 05:47:19 2009
@@ -127,7 +127,7 @@
             <#if webSiteId?exists && requestAttributes._CURRENT_VIEW_?exists>
               <#include "component://common/webcommon/includes/helplink.ftl" />
               <#if helpContent?has_content ||  helpTopic == "navigateHelp" || (parameters.portalPageId?exists && helpTopic == "MYPORTAL_showPortalP")>
-                <span><a href="javascript:lookup_popup2('showHelp?helpTopic=${helpTopic}&portalPageId=${parameters.portalPageId?if_exists}','help' ,500,500);">${uiLabelMap.CommonHelp}</a></span>
+                <span><a href="javascript:lookup_popup2('showHelp?helpTopic=${helpTopic}&amp;portalPageId=${parameters.portalPageId?if_exists}','help' ,500,500);">${uiLabelMap.CommonHelp}</a></span>
               <#else>
                 <span><a href="${helpUrlPrefix}${helpUrlTopic}${helpUrlSuffix}" target="_blank">${uiLabelMap.CommonHelp}</a></span>
               </#if>

Modified: ofbiz/trunk/themes/bizznesstime/includes/secondary-appbar.ftl
URL: http://svn.apache.org/viewvc/ofbiz/trunk/themes/bizznesstime/includes/secondary-appbar.ftl?rev=816856&r1=816855&r2=816856&view=diff
==============================================================================
--- ofbiz/trunk/themes/bizznesstime/includes/secondary-appbar.ftl (original)
+++ ofbiz/trunk/themes/bizznesstime/includes/secondary-appbar.ftl Sat Sep 19 05:47:19 2009
@@ -26,7 +26,7 @@
 
 <#if userLogin?has_content>
   <ul>
-  <h4>${uiLabelMap.CommonSecondaryApps}</h4>
+    <li><h4>${uiLabelMap.CommonSecondaryApps}</h4></li>
     <#list displayApps as display>
       <#assign thisApp = display.getContextRoot()>
       <#assign permission = true>