Review Request: Add a systemsettings option to enable spellchecking by default.

Jesper Thomschutz jesperht at yahoo.com
Mon Jul 27 11:10:46 BST 2009


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1103/
-----------------------------------------------------------

(Updated 2009-07-27 10:10:45.871762)


Review request for kdelibs.


Changes
-------

Getting rid of memory leak as advised by Darktears


Summary
-------

Motivation: I'm freakin' tired of always having to right click a text edit and chose "enable auto spell check". I'm a horrible speller - want to tell KDE to always check my spelling!

This patch adds an checkbox under systemsettings/languages/spellchecker to allow the user to decide if they want to have the autospellchecking feature enabled on their textedits by default or not. The checkbox uses the sonnet config, and uses the key "checkerEnabledByDefault". KTextEdit has been modified to use this value/setting if it is available.


Diffs (updated)
-----

  trunk/KDE/kdelibs/kdecore/sonnet/settings.cpp 1002882 
  trunk/KDE/kdelibs/kdecore/sonnet/settings_p.h 1002882 
  trunk/KDE/kdelibs/kdeui/sonnet/configui.ui 1002882 
  trunk/KDE/kdelibs/kdeui/sonnet/configwidget.cpp 1002882 
  trunk/KDE/kdelibs/kdeui/widgets/ktextedit.cpp 1002882 

Diff: http://reviewboard.kde.org/r/1103/diff


Testing
-------

Ran systemsettings, then launched a ktextedit. The settings from systemsettings are reflected and spellcheck is enabled/disabled.


Thanks,

Jesper





More information about the kde-core-devel mailing list