<div dir="ltr"><div><div>Hi Nitish,<br><br></div>I think we said at one time that we would like to rename this application as it is a great generic application that can be used in many other cases than multiplications.<br><br></div><div>Regards,<br></div><div><br></div><div>Emmanuel<br></div><br></div><div class="gmail_extra"><br><div class="gmail_quote">2017-03-15 21:50 GMT+01:00 JAZEIX Johnny <span dir="ltr"><<a href="mailto:jazeix@gmail.com" target="_blank">jazeix@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  
    
  
  <div bgcolor="#FFFFFF" text="#000000">
    <div class="m_7108133406506594863moz-cite-prefix">Hi,<br>
      <br>
      Your diff is incomplete, it only contains the last commit not the
      whole activity so we can't really make comments on it.<br>
      <br>
      I'm doing some of them here:<br>
      <br>
      * it misses a default value for the configuration. If you remove
      it from the config file (~/.config/gcompris/gcompris-<wbr>qt.conf) and
      start again GCompris, when you go on the configuration, the box
      will be empty.<br>
      <br>
      * Also, the save/load of the configuration does not work. As you
      can see in the log: save configuration for: "" instead of save
      configuration for: "multiplication_tables". It's because you
      didn't initialize the dialogActivityConfig in
      Component.onCompleted. You can check on other activities using
      configuration (traffic).<br>
      <br>
      * Is it useful to have 2 different buttons "Start" and "Finish" or
      can we only have one?<br>
      <br>
      * For the "school" mode, we can put as many questions as we want.
      It's not handled properly in the activity then (questions override
      the different buttons for example).<br>
      <br>
      * shouldn't we change the name of the modes? "School mode" and
      "Normal mode" are not really explicit.<span class="HOEnZb"><font color="#888888"><br>
      <br>
      Johnny</font></span><div><div class="h5"><br>
      <br>
      On 03/10/17 00:34, Emmanuel Charruau wrote:<br>
    </div></div></div><div><div class="h5">
    <blockquote type="cite">
      <div dir="ltr">
        <div>
          <div>
            <div>
              <div>
                <div>
                  <div>
                    <div>
                      <div>
                        <div>Hi Nitish,<br>
                          <br>
                        </div>
                        Just played it without looking at the code.<br>
                      </div>
                      Very nice to play, it looks like I imagined it.<br>
                      <br>
                    </div>
                    Two remarks at the moment.<br>
                  </div>
                  In school mode, the cards should be shuffled to avoid
                  pupils to learn the positions of the answers. I saw
                  them last year learning by heart the sequenced
                  answers. I shuffled the questions and a few pupils
                  where totally lost.<br>
                </div>
                When choosing the mode, the presentation (school mode
                select your mode) lost me. It could be more obvious.<br>
                <br>
              </div>
              Next step for me will be to look at the code to see if we
              can proposer long sentences.<br>
              <br>
            </div>
            Great job :)<br>
            <br>
          </div>
          Thanks<br>
          <br>
        </div>
        Emmanuel<br>
        <div>
          <div>
            <div>
              <div>
                <div> <br>
                </div>
              </div>
            </div>
          </div>
        </div>
      </div>
      <div class="gmail_extra"><br>
        <div class="gmail_quote">2017-03-09 5:45 GMT+01:00 nitish
          chauhan <span dir="ltr"><<a href="mailto:nitish.nc18@gmail.com" target="_blank">nitish.nc18@gmail.com</a>></span>:<br>
          <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
            <div dir="ltr">Hi,<br>
              the current status of the activity can be tracked here :-<br>
              <br>
              <div><b>latest commit :-</b> <a href="https://cgit.kde.org/gcompris.git/commit/?h=sok_nitishc_multiplicationtables&id=8bb92a326f3c805943377f2aa50b7bce0b69b9e1" target="_blank">https://cgit.kde.org/gcompr<wbr>is.git/commit/?h=sok_nitishc_<wbr>multiplicationtables&id=8bb92a<wbr>326f3c805943377f2aa50b7bce0b69<wbr>b9e1</a></div>
              <div><b><br>
                </b></div>
              <div><b>Code review phabricator :-</b> <br>
              </div>
              <div><a href="https://phabricator.kde.org/D4982" target="_blank">https://phabricator.kde.org/D4<wbr>982</a><br>
              </div>
              <div><br>
              </div>
              <div>Task done :-</div>
              <div>1.) Both normal & school mode completed<br>
                2,) Minor bug fixes & code refracting.<br>
                <br>
              </div>
              <div>please share your feedback & review.<br>
                <br>
                regards,<br>
                Nitish</div>
              <div><br>
              </div>
            </div>
          </blockquote>
        </div>
        <br>
      </div>
    </blockquote>
    <p><br>
    </p>
  </div></div></div>

</blockquote></div><br></div>