--===============1516431338== Content-Type: multipart/alternative; boundary="===============3974018130998668377==" --===============3974018130998668377== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101078/#review2572 ----------------------------------------------------------- Ship it! I checked it and it works as expected! Do you want a git account so you can= push this yourself and carry on fixing KTouch? - Anne-Marie On April 10, 2011, 8:49 p.m., Peter R=C3=BCthemann wrote: > = > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/101078/ > ----------------------------------------------------------- > = > (Updated April 10, 2011, 8:49 p.m.) > = > = > Review request for KDE Edu. > = > = > Summary > ------- > = > Because ktouch finally moved to git, I publish my bug fix again. > = > I fixed a bug in KTouch. At the moment it's not possible to clear the sta= tistics in the "training statistics" window. Only a part of the statistics = was cleared. = > = > Changelog: > - The fix clears the current statistics and refreshes the UI > - The sessions restarts at the current level > - The clearStatistics function has now a parameter. The parameter decides= whether a new session should be started or not. > = > I really like git and kdeedu, so I hope I can contribute a little bit. > = > Peter > = > = > This addresses bug 149758. > http://bugs.kde.org/show_bug.cgi?id=3D149758 > = > = > Diffs > ----- > = > src/ktouch.h 3744e72 = > src/ktouch.cpp 55bab64 = > src/ktouchstatisticsdialog.cpp 64faa43 = > = > Diff: http://git.reviewboard.kde.org/r/101078/diff > = > = > Testing > ------- > = > Fix works with compiled KTouch. > = > = > Thanks, > = > Peter > = > --===============3974018130998668377== Content-Type: text/html; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable
This is an automatically generated e-mail. To reply, visit: http://git.revie= wboard.kde.org/r/101078/

Ship it!

I checked =
it and it works as expected! Do you want a git account so you can push this=
 yourself and carry on fixing KTouch?

- Anne-Marie


On April 10th, 2011, 8:49 p.m., Peter R=C3=BCthemann wrote:

Review request for KDE Edu.
By Peter R=C3=BCthemann.

Updated April 10, 2011, 8:49 p.m.

Descripti= on

Because ktouch finally moved to git, I publish my bug fix ag=
ain.

I fixed a bug in KTouch. At the moment it's not possible to clear the s=
tatistics in the "training statistics" window. Only a part of the=
 statistics was cleared. =


Changelog:
- The fix clears the current statistics and refreshes the UI
- The sessions restarts at the current level
- The clearStatistics function has now a parameter. The parameter decides w=
hether a new session should be started or not.

I really like git and kdeedu, so I hope I can contribute a little bit.

Peter

Testing <= /h1>
Fix works with compiled KTouch.
Bugs: 149758

Diffs=

  • src/ktouch.h (3744e72)
  • src/ktouch.cpp (55bab64)
  • src/ktouchstatisticsdialog.cpp (64faa43)

View Diff

--===============3974018130998668377==-- --===============1516431338== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ kde-edu mailing list kde-edu@mail.kde.org https://mail.kde.org/mailman/listinfo/kde-edu --===============1516431338==--