D26712: Multiple Dataset for enumerate

Johnny Jazeix noreply at phabricator.kde.org
Thu Jan 30 07:41:48 GMT 2020


jjazeix added inline comments.

INLINE COMMENTS

> enumerate.js:111
> +            items.answerColumn.itemAt(i).color = "red";
> +            items.okButton.enabled = false;
> +        }else

why is it in the for loop and not outside, at the start of the function directly?
This should not be used for both disabling the button and check if the answer is correct, there should be another variable for the latter.

REPOSITORY
  R2 GCompris

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

To: shubhammishra, #gcompris_improvements, jjazeix, echarruau
Cc: johnny, kde-edu, narvaez, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20200130/060b7dc0/attachment.html>


More information about the kde-edu mailing list