<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/D17945">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
color: #6B748C;
font-style: italic;
margin: 4px 0 12px 0;
padding: 8px 12px;
background-color: #F8F9FC;">
<div style="font-style: normal;
padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D17945#412138" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D17945#412138</a>, <a href="https://phabricator.kde.org/p/echarruau/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@echarruau</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><p>6- tutorial page 4<br />
Replace "No, this is a wrong anwer by: Your made an error: When divided by 2, 13 is equal to 6 and the remainder is 1. Therefore this is an odd number.<br />
All this must fit in the box.</p></div>
</blockquote>
<p>What do you think about using AnswerButton (the same as readingh/readingv/leftright/missing-letter activities) and avoid adding the text: "Great" or "Error"?</p>
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>7- tutorial page 4<br />
Please center the great in the box.<br />
8- tutorial page 5<br />
Same comments that for points 6 and 7.<br />
9- NumbersOddEven.qml content should be moved to directory ressources<br />
10- NumbersOddEven.qml tutorialInstructions should be dispatched in tutorialsx.qml files.</p></blockquote>
<p>I would let them in the NumbersOddEven file as they are directly inherited from the parent Item, else it would mean add more variables to get the path of the instructions for not a big gain.</p></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/D17945">https://phabricator.kde.org/D17945</a></div></div><br /><div><strong>To: </strong>dekumar, GCompris: Improvements, jjazeix<br /><strong>Cc: </strong>echarruau, amankumargupta, jjazeix, kde-edu, narvaez, apol<br /></div>