Railroad review

[Johnny Jazeix]

Hi,

it may be due to the "import QtQuick.Controls 2.0" that may be not
available in the Qt version I used to compile the apk. I'll try to
regenerate one tonight now that it is removed.

Johnny


2017-04-12 15:07 GMT+02:00 Utkarsh Tiwari <iamutkarshtiwari at gmail.com>:

> Hi,
>
> I am facing an issue with the railroad apk. On clicking the Railroad icon,
> nothing happens. Though the apk works fine for all other activities, it's
> just the Railroad activity that doesn't seem to start. What could be the
> possible issue?
>
> Regards,
> Utkarsh Tiwari
>
>
> On Thu, 6 Apr 2017 at 16:03 Utkarsh Tiwari <iamutkarshtiwari at gmail.com>
> wrote:
>
>> 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/20170412/1cafd6d1/attachment-0001.html>