In R2:e58278c59b86968e8474ccbba4a94ce3fa0f8bae#2004440, @jjazeix wrote:

we can still play after the game is won.
When it's the opponent turn and you click on your beans, there is the animation playing and causing the beans to go to the left side of the house.

Does that mean the shaking of beans? I added it to let the user know that it is not their turn. Should it be removed?

The animation is not "good" enough. We should see the move house by house (if possible when adding a bean and if they are won, when we remove them)

Yes, but it's possible to have an animation for each seed? It's added to the image of a single grain only but I am not sure if we can do for single seed?

In R2:26d4f0ab471532aa169d872f772ff04a735a5dca#2004083, @jjazeix wrote:

In R2:26d4f0ab471532aa169d872f772ff04a735a5dca#2004071, @dmadaan wrote:

In R2:26d4f0ab471532aa169d872f772ff04a735a5dca#2004067, @jjazeix wrote:

You still haven't commited tutorial3.png image!

Yes, I still need to make and edit it. The lack of skills of inkscape and gimp made me lazy :D :)

Put a default one, we will improve it later

Before starting the AI, I think it would be better to work on the animations (moving of the seeds).
If I play a game (not necessarely finish it), I leave the activity and go back, the scores are set to 0. But once I play the second player, its score goes back to the previous one (same for second turn for player 1).

Sorry, I didn't understand this one. The previous means the previous score? How can I reproduce it?

Play, leave the activity, play again, see the scores (captured seeds).

I saw it, will try fixing it in next commit :)

Reload or winning does not reset the score.
In two player mode, no need of levels.
The number of seeds is hidden by the image, we can display them better.
Are the buttons really needed? Can it be easier to just click on the bowl we want to empty? This way, we can use the space below and above to display the seed count.

Yes, I think the buttons are not really needed. IMO the first intution the child will get is clicking the houses itself rather than the buttons :) I am not sure if seed count that large would look good?

No need for it to be large, just need to be below/above the good house.

Also, I forgot to mention, but in vertical mode it may be better to have the oware rotated to better fit the height.

I asked about it in one of my previous comments ;) I had a doubt that should everything be rotated or just the board and keep rest i.e the scores etc the same or make it just a one mode activity?

In R2:26d4f0ab471532aa169d872f772ff04a735a5dca#2004067, @jjazeix wrote:

You still haven't commited tutorial3.png image!

Yes, I still need to make and edit it. The lack of skills of inkscape and gimp made me lazy :D :)

Before starting the AI, I think it would be better to work on the animations (moving of the seeds).
If I play a game (not necessarely finish it), I leave the activity and go back, the scores are set to 0. But once I play the second player, its score goes back to the previous one (same for second turn for player 1).

Sorry, I didn't understand this one. The previous means the previous score? How can I reproduce it?

Reload or winning does not reset the score.
In two player mode, no need of levels.
The number of seeds is hidden by the image, we can display them better.
Are the buttons really needed? Can it be easier to just click on the bowl we want to empty? This way, we can use the space below and above to display the seed count.

Yes, I think the buttons are not really needed. IMO the first intution the child will get is clicking the houses itself rather than the buttons :) I am not sure if seed count that large would look good?

1.For instructions, I have used an array but the problem with it probably we can't use qsTr with it? We have a better way?

2. For vertical mode, along with board everything needs to be rotated? Will it just be horizontal activity then?

1. The url while leaving the config is bad. It gives the error: Error downloading Contents from QUrl("http://gcompris.net/Contents") : QNetworkReply::NetworkError(ContentNotFoundError) : "Error transferring http://gcompris.net/Contents - server replied: Not Found"
2. The music should be played after the welcome sound.
3. Weird delay of music was observed on activation and deactivation from config.

In D5891#112054, @jjazeix wrote:

Is it normal to be able to change the levels (in expert mode we can't)?

Level is the array size. In expert mode (that we had before) there was just one level randomly selected. Now its a cetegory with all its levels so its normal in this case to have the level numbers. The point would be to display it similar to as we do in expert mode? (hardcode the level in expert? )

Would it simplify the code if instead of having the 4 possibilities, we just display 2 comboboxes to the user:

Would it be ok to have 2 combo boxes and let them make their own combinations? As currently we have made and decided of what is required? Also we would require three in that case (one for images also).

• Instructions visible
• Score visible and we handle it with 2 separate booleans in the code? Because for now, we have to know the correspondance between "easy", "medium" and "expert" modes and what is current displayed

I already have booleans corresponding to them. I am not sure if using the booleans directly instead of correspondence will make it easy?

The new mode selects a random category now and user can play the levels of the random category. If user wants to select a new category they can return to menu then click on "I am ready" again.

From the discussion in IRC:
A child is unable to create a notion in just one level. So we need the same think to be done in multiple levels of the same category. A category can be chosen randomly from all the categories and its levels are displayed in continuous. If they are bored they can either skip the levels or go to home and chose another category.

Hi,
I have followed this principle as mentioned:
Startup
if not-first-run

if backgroundMusic is enabled
    if background-music-resource is downloaded
        possibly update and register it
    else 
       download and register it

Overall I liked the concept :) For the coloring of letters, I thought may be color the letter for initial levels (when child is new to recognizing) and not display them in later levels or not highlight the letters at all (its an hint that its the correct word and makes it easy to recognise, is not much learning if the child gets the trick)? It can be an increase in difficulty. I am not sure if its a good idea but it is to be discussed if it is required and if it can be an improvement?

Yes, the main aim here would be to have the same letter displayed again if the user guesses the wrong words instead of displaying a new letter to find. There can be a validation which checks if all the correct words are selected, if not then display the letter again.