Review Request 122674: New option to ignore punctuations

ankit wagadre wagadre.ankit at gmail.com
Sun Feb 22 19:40:25 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122674/
-----------------------------------------------------------

(Updated Feb. 22, 2015, 7:40 p.m.)


Review request for KDE Edu and Inge Wallin.


Repository: parley


Description
-------

option to ignore punctuations.check this one i'll delete old one.


Diffs
-----

  src/practice/writtenpracticevalidator.cpp e974441 
  src/practice/configure/generalpracticeoptions.ui a40c739 
  src/settings/parley.kcfg 0037420 
  src/practice/writtenpracticevalidator.h 93e7423 
  src/practice/writtenbackendmode.cpp 7b221b7 
  src/collection/testentry.h 4f47c38 

Diff: https://git.reviewboard.kde.org/r/122674/diff/


Testing
-------

i have done testing it works fine.and about ur comment that y should capitaliztion check for punctuation,if u ingnore capitalization is checked then the string should be checked for puntuation mistake for lower alphabets.


File Attachments (updated)
----------------

0002-punctuation-option-correction.patch
  https://git.reviewboard.kde.org/media/uploaded/files/2015/02/22/3783052f-3d7b-446c-85ca-0b205b863d8b__0002-punctuation-option-correction.patch


Thanks,

ankit wagadre

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20150222/5d06acf4/attachment.html>


More information about the kde-edu mailing list