----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125532/#review86440 -----------------------------------------------------------
For me seems good as I created same patch in kdepim :) - Laurent Montel On oct. 5, 2015, 3:40 après-midi, Daniel Vrátil wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/125532/ > ----------------------------------------------------------- > > (Updated oct. 5, 2015, 3:40 après-midi) > > > Review request for KDE Frameworks and Laurent Montel. > > > Repository: ktextwidgets > > > Description > ------- > > This patch makes KTextEdit read the default Sonnet settings from the new > location that Sonnet uses (~/.config/KDE/Sonnet.conf instead of > ~/.config/sonnetrc) - IMO there should be a better API to find out the > default value, because having to construct Sonnet::Highlighter just to find > out Sonnet is off by default feels a bit clumsy :) (but that's offtopic for > this RR). > > Additionally this patch adds a new submenu to the editor's context menu that > allows to switch between language used for spellchecking. > > The submenu code comes from PimCommon::CustomTextEditor in kdepim.git. > > I will commit as two separate commits. > > > Diffs > ----- > > src/widgets/ktextedit.cpp f549825 > > Diff: https://git.reviewboard.kde.org/r/125532/diff/ > > > Testing > ------- > > Switched KNoteEditor in KNotes in KDE PIM (which uses > PimCommon::CustomTextEditor) to use this patched KTextEdit, everything works > as before. > > > Thanks, > > Daniel Vrátil > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
