svn commit: r934180 - in /ofbiz/trunk/themes: flatgrey/includes/header.ftl multiflex/includes/header.ftl

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

svn commit: r934180 - in /ofbiz/trunk/themes: flatgrey/includes/header.ftl multiflex/includes/header.ftl

buscob
Author: buscob
Date: Wed Apr 14 20:28:11 2010
New Revision: 934180

URL: http://svn.apache.org/viewvc?rev=934180&view=rev
Log:
A patch from Blas Rodriguez Somoza
OFBIZ-3675 - XHTML validation errors round 2 (themes)
https://issues.apache.org/jira/browse/OFBIZ-3675

XHTML validation errors

    * Added alt attribute to img tag
    * Uppercase tags or attributes.


Modified:
    ofbiz/trunk/themes/flatgrey/includes/header.ftl
    ofbiz/trunk/themes/multiflex/includes/header.ftl

Modified: ofbiz/trunk/themes/flatgrey/includes/header.ftl
URL: http://svn.apache.org/viewvc/ofbiz/trunk/themes/flatgrey/includes/header.ftl?rev=934180&r1=934179&r2=934180&view=diff
==============================================================================
--- ofbiz/trunk/themes/flatgrey/includes/header.ftl (original)
+++ ofbiz/trunk/themes/flatgrey/includes/header.ftl Wed Apr 14 20:28:11 2010
@@ -96,7 +96,7 @@ under the License.
       <#if (userPreferences.COMPACT_HEADER)?default("N") == "Y">
         <li class="logo-area">
           <#if shortcutIcon?has_content>
-            <a href="<@ofbizUrl>${logoLinkURL}</@ofbizUrl>"><img src="<@ofbizContentUrl>${StringUtil.wrapString(shortcutIcon)}</@ofbizContentUrl>"/></a>
+            <a href="<@ofbizUrl>${logoLinkURL}</@ofbizUrl>"><img src="<@ofbizContentUrl>${StringUtil.wrapString(shortcutIcon)}</@ofbizContentUrl>" alt=""/></a>
           </#if>
         </li>
         <#if layoutSettings.topLines?has_content>
@@ -118,7 +118,7 @@ under the License.
           <p class="collapsed">
             <a href="<@ofbizUrl>logout</@ofbizUrl>">${uiLabelMap.CommonLogout}</a>&nbsp;&nbsp;
               <a href="javascript:document.setUserPreferenceCompactHeaderN.submit()">&nbsp;&nbsp;</a>
-              <form name="setUserPreferenceCompactHeaderN" method="post" action="<@ofbizUrl>setUserPreference</@ofbizUrl>" onSubmit="javascript:submitFormDisableSubmits(this)">
+              <form name="setUserPreferenceCompactHeaderN" method="post" action="<@ofbizUrl>setUserPreference</@ofbizUrl>" onsubmit="javascript:submitFormDisableSubmits(this)">
                 <input name="userPrefGroupTypeId" value="GLOBAL_PREFERENCES" type="hidden"/>
                 <input name="userPrefTypeId" value="COMPACT_HEADER" type="hidden"/>
                 <input name="userPrefValue" value="N" type="hidden"/>

Modified: ofbiz/trunk/themes/multiflex/includes/header.ftl
URL: http://svn.apache.org/viewvc/ofbiz/trunk/themes/multiflex/includes/header.ftl?rev=934180&r1=934179&r2=934180&view=diff
==============================================================================
--- ofbiz/trunk/themes/multiflex/includes/header.ftl (original)
+++ ofbiz/trunk/themes/multiflex/includes/header.ftl Wed Apr 14 20:28:11 2010
@@ -37,10 +37,10 @@ under the License.
         <!-- Navigation Level 0 -->
         <div class="nav0">
           <ul>
-            <li><a href="<@ofbizUrl>setSessionLocale?newLocale=it</@ofbizUrl>"><img src="/multiflex/flag_it.gif" /></a></li>
-            <li><a href="<@ofbizUrl>setSessionLocale?newLocale=en</@ofbizUrl>"><img src="/multiflex/flag_en.gif" /></a></li>
-            <li><a href="<@ofbizUrl>setSessionLocale?newLocale=de</@ofbizUrl>"><img src="/multiflex/flag_de.gif" /></a></li>
-            <li><a href="<@ofbizUrl>setSessionLocale?newLocale=fr</@ofbizUrl>"><img src="/multiflex/flag_fr.gif" /></a></li>
+            <li><a href="<@ofbizUrl>setSessionLocale?newLocale=it</@ofbizUrl>"><img src="/multiflex/flag_it.gif" alt="" /></a></li>
+            <li><a href="<@ofbizUrl>setSessionLocale?newLocale=en</@ofbizUrl>"><img src="/multiflex/flag_en.gif" alt="" /></a></li>
+            <li><a href="<@ofbizUrl>setSessionLocale?newLocale=de</@ofbizUrl>"><img src="/multiflex/flag_de.gif" alt="" /></a></li>
+            <li><a href="<@ofbizUrl>setSessionLocale?newLocale=fr</@ofbizUrl>"><img src="/multiflex/flag_fr.gif" alt="" /></a></li>
           </ul>
         </div>