Hi Scott,

Yes, I thought the same. Then I searched in *.ftl and find 584 occurences of the same type (style="). And I thought also that adding a style for this only case was a bit too much... Of course if there is already a such style I'm ready to improve. Or if we knew where there should the same be apllied, then we could generalize...

Jacques

From: "Scott Gray" <scott.g...@hotwaxmedia.com>
Hi Jacques

http://docs.ofbiz.org/display/OFBADMIN/HTML+and+CSS+Best+Practices
"HTML should be well structured, concise, and free of styling information. Well structured HTML is easily styled with style sheets (CSS) - therefore all styling code should be kept in the style sheets."

Regards
Scott

HotWax Media
http://www.hotwaxmedia.com
801.657.2909

----- Original Message -----
From: jler...@apache.org
To: comm...@ofbiz.apache.org
Sent: Saturday, February 7, 2009 12:58:29 PM GMT -07:00 US/Canada Mountain
Subject: svn commit: r741948 - 
/ofbiz/trunk/applications/order/webapp/ordermgr/order/findOrders.ftl

Author: jleroux
Date: Sat Feb  7 19:58:28 2009
New Revision: 741948

URL: http://svn.apache.org/viewvc?rev=741948&view=rev
Log:
A patch from Sumit Porwal "On find orders form, user can not select multiple Role Types however the "findOrders" service supports it." (https://issues.apache.org/jira/browse/OFBIZ-2136) - OFBIZ-2136

Modified:
   ofbiz/trunk/applications/order/webapp/ordermgr/order/findOrders.ftl

Modified: ofbiz/trunk/applications/order/webapp/ordermgr/order/findOrders.ftl
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/webapp/ordermgr/order/findOrders.ftl?rev=741948&r1=741947&r2=741948&view=diff
==============================================================================
--- ofbiz/trunk/applications/order/webapp/ordermgr/order/findOrders.ftl 
(original)
+++ ofbiz/trunk/applications/order/webapp/ordermgr/order/findOrders.ftl Sat Feb 
 7 19:58:28 2009
@@ -132,7 +132,7 @@
                <td width='25%' align='right' 
class='label'>${uiLabelMap.PartyRoleType}</td>
                <td width='5%'>&nbsp;</td>
                <td align='left'>
-                  <select name='roleTypeId'>
+                  <select name='roleTypeId' multiple="multiple" 
style="height:5em">
                    <#if currentRole?has_content>
                    <option 
value="${currentRole.roleTypeId}">${currentRole.get("description", 
locale)}</option>
                    <option value="${currentRole.roleTypeId}">---</option>




Reply via email to