Re: [PythonReports-users] Patch for editor save and save-as options

[alexander s. <al...@go...>]

On 08.12.2012 0:01, Werner F. Bruhin wrote:
>
>> Text diff is fine, just don't use HTML in emails.
>
> Noted, and I set you and the list address as "text" recipient -
> but not sure this takes - is this one coming in as html for you?

This one is plain text.

> Do you consider the WIT and the Persist stuff too, or do you have an
> issue with them - no problem either way.

Yes, I do consider, and yes, I have issues.

For SetState fix, I don't understand why this method should not be 
called in user code.  I don't know why such calls are needed, either. So 
I'm waiting when I meet with Den to ask him how this should be handled.

For Widget Inspection Tool, I understand that it is useful for UI 
hacking, but I don't see it needed in production application.  It's like 
debug prints: you insert print statements all around your code, find 
your bug, and then remove those prints.  Perhaps it could be added with 
InitInspection call hidden under "if False", but I did not made my mind 
about that yet.  Usually unneeded imports are not good for application 
size and, mainly, for start-up times.

For element size change, there is a note that MinSize has issues.  I'd 
like to hear a comment from Den, too.  Anyway, 50 is too big: there are 
certain uses for elements with height near zero.

I have checked in font family fix.

For persistence, I have just looked through it more carefully, and I 
thing that will do.  I'd probably add that after looking through code 
yet another time.

> Have a nice weekend - doing "confit de canard" over the next couple of
> days - so less time for computers:).

Wow!  Does it take several days of intensive cooking?

Cheers,
alex.