<table><tr><td style="">jjazeix added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D21461">View Revision</a></tr></table><br /><div><div><p>This is good start :).<br />
I did some small comments. Please check with your mentors how you prefer to work: either work by diffs, or you create your own branch (do commits when you want) and once it's good create a diff of the set of commits you want to check.</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D21461#inline-120541">View Inline</a><span style="color: #4b4d51; font-weight: bold;">moneyConstants.js:47</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"><span style="color: #aa4000">var</span> <span style="color: #004012">moneyItems</span> <span style="color: #aa2211">=</span> <span class="p">{</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span style="color: #004012">MONEY_EURO_COIN_1C</span><span style="color: #aa2211">:</span>   <span class="p">{</span> <span style="color: #004012">img</span><span style="color: #aa2211">:</span> <span style="color: #766510">"c1c.svg"</span><span class="p">,</span>  <span style="color: #004012">val</span><span style="color: #aa2211">:</span> <span style="color: #601200">0.01</span> <span class="p">},</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span style="color: #004012">MONEY_EURO_COIN_2C</span><span style="color: #aa2211">:</span>   <span class="p">{</span> <span style="color: #004012">img</span><span style="color: #aa2211">:</span> <span style="color: #766510">"c2c.svg"</span><span class="p">,</span>  <span style="color: #004012">val</span><span style="color: #aa2211">:</span> <span style="color: #601200">0.02</span> <span class="p">},</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">This is more of an improvement but can you put here the full path to the images?<br />
This way, if someone wants to make a dataset with external files (without embed images in rcc), it will still be possible?</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D21461#inline-120542">View Inline</a><span style="color: #4b4d51; font-weight: bold;">Data.qml:27</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"><span style="color: #004012">Dataset</span> <span class="p">{</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span style="color: #aa4000">objective:</span> <span style="color: #004012">qsTr</span><span class="p">(</span><span style="color: #766510">"Age Group 3 - 4 years"</span><span class="p">)</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span style="color: #004012">property</span> <span style="color: #aa4000">var</span> <span style="color: #aa4000">moneyItems:</span> <span style="color: #004012">Constants</span><span class="p">.</span><span style="color: #004012">moneyItems</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Is it possible to have a better objective text? The aim is for the child to know what they will learn or what will the dataset contains. For example, "Learn how to pay up to 10 euros (using money between 1 euro and 5 euros)." (last part may be unnecessary).</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R2 GCompris</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D21461">https://phabricator.kde.org/D21461</a></div></div><br /><div><strong>To: </strong>AkshayCHD, GCompris: Improvements<br /><strong>Cc: </strong>jjazeix, kde-edu, GCompris: Improvements, sanjayshetty, parimalprasoon, harrymecwan, ganeshredcobra, nityanandkumar, echarruau, andreask, rahulyadav, narvaez, scagarwal, apol, timotheegiet, bcoudoin<br /></div>