Ed Leafe wrote:
> On Jun 19, 2008, at 1:46 PM, Paul McNett wrote:
> 
>> You removed the ability for appdev code to position the dialog sizer,
>> resulting in a broken layout:
> 
> 
>       OK, I thought about this before removing it, because it's using the  
> ui-specific positioning, and I couldn't find any examples that, well,  
> used a layout like you have in that form. I'll see about adding that  
> back in.

Thanks... screen real-estate is at a premium, and those buttons taking 
up an entire row just doesn't fly, at least in this case.


>> Also, I'm getting these when I click OK or Cancel:
>>
>> TypeError: onOK() takes exactly 2 arguments (1 given)
>> TypeError: onCancel() takes exactly 2 arguments (1 given)
>>
>> This is because I'd followed the original instructions of overriding
>> those hook methods that are now event handlers. I can easily go into  
>> my
>> code and add the evt arg, but I thought I'd mention this.
> 
>       These aren't event handlers; they are methods that are called by the  
> respective handlers. Perhaps the on* names are confusing? Can you  
> suggest a better naming scheme?

Ah, they originally were event handlers, but now they are called by the 
event handlers. I think on*() is good, but it is unfortunate this is 
likely to break existing code.


Paul


_______________________________________________
Post Messages to: [email protected]
Subscription Maintenance: http://leafe.com/mailman/listinfo/dabo-dev
Searchable Archives: http://leafe.com/archives/search/dabo-dev
This message: http://leafe.com/archives/byMID/[EMAIL PROTECTED]

Reply via email to