Thanks for the pointer Scott!

Done in revision: 741964
I checked the only other mutiple select (for *.ftl) is in Product/QuickAdmin, and this change solves also an issue there (visible before this commit in FF3 also)

Jacques

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

I don't think 584 occurrences of a practice which is agreed by all to be bad 
makes a good argument for having 585 :-)

Besides it should just be a matter of adding something like:
select[multiple="multiple"] {
height: 5em;
}

below the current select selector in maincss.css

Regards
Scott

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


----- Original Message -----
From: "Jacques Le Roux" <jacques.le.r...@les7arts.com>
To: dev@ofbiz.apache.org
Sent: Saturday, February 7, 2009 1:18:06 PM GMT -07:00 US/Canada Mountain
Subject: Re: svn commit: r741948 - 
/ofbiz/trunk/applications/order/webapp/ordermgr/order/findOrders.ftl

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