Re: Testers required

Dietmar Schwertberger <[email protected]>
Newsgroups gmane.comp.python.wxglade
Message-ID <[email protected]>
On 23.09.2016 04:52, Udi Fuchs wrote:
> Here are a few issues with today's code from 
> https://bitbucket.org/wxglade/wxglade.
Please note that the full modifications are in DEV_DS_2016. The default 
branch is a bit behind.
Your first obervations were fixed in DEV... already.


> 1. The program does not start because args point to files on your 
> system. Fixed with args.patch
Yes, sorry, Hg lacks a permanent marking for code which should not be 
committed. So sometimes test code makes it into the repository...

> 3. Trying to modify TextCtrl font crashes. Fixed with font.patch
Thanks. I've added your patch to the DEV branch.

>
> 4. Text control fonts are saved to wxg, but wxglade ignores the font 
> when opening file, with no error message. Can easily be reproduced by 
> changing the font in tests/casefiles/FontColour.wxg
OK, I will have a look. I think I have not yet done anything with fonts...

> I also noticed that the order of xml elements in wxg files changed. It 
> is a bit annoying when trying to figure out what changed in a wxg 
> file. I'm not sure what determines the order, but it would be nice if 
> this order could be locked down to something deterministic.
The previous code did not have a defined order.
The new version should write in the ordering that is defined in the class.
E.g. EditTextCtrl:
      _PROPERTIES = ["Widget", "value", "style"]
     PROPERTIES = ManagedBase.PROPERTIES + _PROPERTIES + 
ManagedBase.EXTRA_PROPERTIES
So the full ordering here is:
  ['name', 'class', 'custom_base', 'id', 'size', 'background', 
'foreground', 'font', 'tooltip', 'disabled', 'focused', 'hidden', 'pos', 
'proportion', 'border', 'flag', 'value', 'style', 'events', 'extracode', 
'extraproperties']

I have been using Altova XML Spy for the diffs as I have this available 
at work. The Altova products can be downloaded and tested for 30 days. 
Unfortunately I could not find a free tool.



> That's it for today,
Thanks a lot,

Dietmar


------------------------------------------------------------------------------
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.