Railroad review

[JAZEIX Johnny]

Hi,

I tested it and added comments in 
https://phabricator.kde.org/R2:fbf51ef721fc50e1b7d82bda75dfc0aa19ceed32 
directly.

Johnny

On 03/12/17 22:20, JAZEIX Johnny wrote:
> Hi,
>
> I didn't test it but here is the apk: 
> https://drive.google.com/open?id=0B1Wylu9YXKcKaVVDUmVMV3E0Tmc
>
> For reload, I didn't say it's a bad idea :). It would be even better 
> if once you click on reload, it displays the initial wagons and if you 
> click again, you don't restart from beginning but from where you were. 
> And in this case, it's no more a "reload" but more a "hint". I just 
> checked and what I'm talking about is the current implementation as in 
> the Gtk+ version if you want to test.
>
> Johnny
>
> On 03/10/17 17:27, Utkarsh Tiwari wrote:
>> Hi Johnny,
>> I agree with your opinion on timer, it's adds stress factor. We 
>> should remove it.
>> Reload button sounds like a good idea instead of restarting the game 
>> everytime.
>>
>> According to me, the horizontal layout would appear better than 
>> vertical layout for flows. Putting all the wagon in a row would still 
>> need to add the scroll view. Swiping with flick is easier on mobile 
>> devices than smaller scroll bars. If you would suggest the vertical 
>> layout to be implemented, I can do that :)
>>
>> Could you please provide me with the apk of the latest Railroad code? 
>> I would like to test it on my android device.
>>
>> Regards,
>> Utkarsh Tiwari
>>
>> On Thu, 9 Mar 2017 at 02:32 JAZEIX Johnny <jazeix at gmail.com 
>> <mailto:jazeix at gmail.com>> wrote:
>>
>>     Hi,
>>
>>     for me the timer has no plus-value (except potentially stress
>>     children if they don't manage to memorize in time) so I would
>>     remove it.
>>
>>     I would also change the reload button to the hint button that
>>     would display the locomotives again (instead of removing
>>     everything) and potentially hiding all the locomotives below
>>     while it's on hint mode.
>>
>>     You can probably add one step in the introduction to tell the
>>     child to drag and drop the wagons on top.
>>
>>     I find a little confusing having the drag/drop and swiping with
>>     the same command (basically, swiping if you select and go
>>     left/right and drag item if going top or bottom?). Maybe if the
>>     flows were top to bottom (meaning more rows, all the wagons in
>>     one row being always displayed) with a vertical scrollbar on the
>>     side, it would be better? What do you think?
>>
>>     I think resource/bg.svg is not used.
>>
>>     Except these points the logic is good for me, I'll check the code
>>     later.
>>
>>
>>     Johnny
>>
>>
>>     On 03/05/17 14:55, Utkarsh Tiwari wrote:
>>>     Hi,
>>>         Here is the updated Revision for the Railroad activity -
>>>     https://phabricator.kde.org/D4014
>>>
>>>     Improvements:
>>>
>>>      1. Drag/drop improved for top and lower section
>>>      2. Timer added to let player have time to memorize and do swipe
>>>         view before the train vanishes
>>>      3. Intro message changed
>>>      4. Swipe area provide in the top carriages section for user to
>>>         view hidden ones in portrait mode
>>>
>>>
>>>     Please test it and share your reviews
>>>
>>>     Regards,
>>>     Utkarsh Tiwari
>>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/gcompris-devel/attachments/20170404/70689e1c/attachment.html>