[gcompris-devel] Re: Three new python boards

[Christof Petig]

Bruno Coudoin schrieb:
> Very good. That's a usual activity for the kids. Good to have and not
> that complex, just 184 line of code ;)
> In the code I found the variable zufall. doesn't seems british... please
> avoid to use non english in the code to help the maintainers.

escaped my review (patch appended)

>>- minouche: find the strawberry hidden (color codes distance)
>>- hexagon: the hexagonal variant of minouche (I particularly like this 
>>board, it looks very pretty) [RPGers like hexagonal maps for their easy 
>>distance measurement]
>>
> 
> I saw that there is 2 different file for each activity but to me it
> should be a level 1 = minouche, level 2 = hexagon. Do you see a way to
> do that or do you plan to add levels to each one ? For example, would
> the concept work with letters and numbers. 

Merging both activities into a two level one we thought about, too. Some 
reasons stood against

- enhancing and unificating the existing activity was beyond the skill 
of the programmer that time. This way the minouche activity remained 
easy to understand (if you look at the code).

- supporting a hexagonal grid modified most of the program in subtle 
ways. If you merge the activities it will be more difficult to 
understand the program. We actually thought of dumping the rectangular 
one once the more pretty hexagonal was done. This option still exists!

- I wanted to get feedback on whether the work would be needed at all.

>>PPS: I did not manage to get semitransparent rects (is that correct)
> 
> 
> You can play with this with the anti aliased canvas (gcompris -a). I
> added transparency to the hexagon and manouche items.

eh, I forgot about that one ... I thought AA would get standard some 
time ...

   Christof

PS: Is it only me seeing no activity on the list (perhaps a virus nuked 
my subscription to most of the lists)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: zufall.patch
Type: text/x-patch
Size: 1640 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/gcompris-devel/attachments/20040402/723b624e/attachment.patch>