D21710: Add multiple datasets to smallnumbers activity

[Akshay Kumar]

AkshayCHD marked 5 inline comments as done.
AkshayCHD added inline comments.

INLINE COMMENTS

> scagarwal wrote in Gletters.qml:89
> Do not use `undefined` . Please use `null`

@scagarwal I've made the changes here https://phabricator.kde.org/D21841

> amankumargupta wrote in Data.qml:26
> "Select a number on the dice upto ..." . Same comments for the ones below.

@amankumargupta made the changes here https://phabricator.kde.org/D21841

> echarruau wrote in Data.qml:30
> On a pedagogical point of view we absolutly need here an objective per level.
> Pupils have to know what they are learning. The main goal is not enough.
> We should have an objective within data: :[ displayed by a message before to start any new level.

@echarruau I have implemented objective text per level you may have a look at it here https://phabricator.kde.org/D21841

REPOSITORY
  R2 GCompris

REVISION DETAIL
  https://phabricator.kde.org/D21710

To: AkshayCHD, #gcompris_improvements, timotheegiet, scagarwal
Cc: scagarwal, amankumargupta, echarruau, timotheegiet, jjazeix, #gcompris_improvements, kde-edu, #gcompris, sanjayshetty, parimalprasoon, harrymecwan, ganeshredcobra, nityanandkumar, andreask, rahulyadav, narvaez, apol, bcoudoin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20190622/d9b922ec/attachment-0001.html>