Just curious to know what should be the expected behavior when a user select "Any Role Type" along with any other specific Roles in Role Type select box ?

Vikas

On Feb 8, 2009, at 2:55 AM, Jacques Le Roux wrote:

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>






Attachment: smime.p7s
Description: S/MIME cryptographic signature

Reply via email to