Railroad review

[Utkarsh Tiwari]

Hi Johnny,

Here is the updated revision - https://phabricator.kde.org/D4014

Thank you,
Utkarsh Tiwari

On Wed, 5 Apr 2017 at 02:00 JAZEIX Johnny <jazeix at gmail.com> wrote:

> 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> 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
>
>
>
> --
Regards,
Utkarsh Tiwari
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/gcompris-devel/attachments/20170406/60bd421e/attachment-0001.html>