<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><p>Can you also please do the other changes (files that shouldn't be updated, typos...)?</p>

<p>Thank you</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/D17945#inline-104218">View Inline</a><span style="color: #4b4d51; font-weight: bold;">TutorialBase.qml:65</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">onClicked:</span> <span class="p">{</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">                         <span style="color: #aa4000">if</span><span class="p">(</span><span style="color: #004012">order</span> <span style="color: #aa2211">===</span> <span style="color: #601200">3</span> <span style="color: #aa2211">||</span> <span style="color: #004012">order</span> <span style="color: #aa2211">===</span> <span style="color: #601200">4</span><span class="p">)</span> <span class="p">{</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">                              <span style="color: #004012">message</span><span class="p">.</span><span style="color: #004012">text</span> <span style="color: #aa2211">=</span> <span style="color: #004012">qsTr</span><span class="p">(</span><span style="color: #766510">"Great"</span><span class="p">)</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">this is not a maintainable way to check this. Let's assume we want to add a new question, we would think that we only need to add a new Tutorial like Tutorial5 but then we will probably forget to update here.<br />
It would be better to have a boolean telling if we want to find the odd or even number at TutorialBase level, and use it to both set the text and tell if it is good or not on click.</p>

<p style="padding: 0; margin: 8px;">Both button should share the same code (there is AnswerButton that could maybe be used? It also have user feedback).</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/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>