You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by Bilgin Ibryam <bi...@gmail.com> on 2009/12/18 14:21:34 UTC

Re: Ajaxifying lookup fields

> Hi Bilgin,
>
> Thanks for your comments, I'm not actually against your approach and 
> think it is a good idea to reuse existing search functionalities.  
> I've had a quick look at your patch (finally) and I have to agree that 
> you're solution is better than mine.  About the verbosity my inline 
> event, I did that on purpose to show the concept in a single file, the 
> actual intention is that you would call minilang or a service to 
> process the event.  But anyway, there are a few things that I think we 
> need to improve in your patch, I'll try and find some time tomorrow to 
> gather my thoughts and put some comments in jira.
>
> Regards
> Scott
Hi Scott,

could you write down what needs to be improved in OFBIZ-3211 whenever 
you got some free time. I'm planning to work on this issue these days.

Thanks
Bilgin

Re: Ajaxifying lookup fields

Posted by Scott Gray <sc...@hotwaxmedia.com>.
On 19/12/2009, at 2:21 AM, Bilgin Ibryam wrote:

>
>> Hi Bilgin,
>>
>> Thanks for your comments, I'm not actually against your approach  
>> and think it is a good idea to reuse existing search  
>> functionalities.  I've had a quick look at your patch (finally) and  
>> I have to agree that you're solution is better than mine.  About  
>> the verbosity my inline event, I did that on purpose to show the  
>> concept in a single file, the actual intention is that you would  
>> call minilang or a service to process the event.  But anyway, there  
>> are a few things that I think we need to improve in your patch,  
>> I'll try and find some time tomorrow to gather my thoughts and put  
>> some comments in jira.
>>
>> Regards
>> Scott
> Hi Scott,
>
> could you write down what needs to be improved in OFBIZ-3211  
> whenever you got some free time. I'm planning to work on this issue  
> these days.
>

Looking at it now, sorry for the delay.

Regards
Scott