--===============5761409103842317774== Content-Type: multipart/alternative; boundary="===============0027531306581157332==" --===============0027531306581157332== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112154/#review38137 ----------------------------------------------------------- src/editor/editor.cpp Hmm, seems like an extra newline, yes. I'll check it. - Inge Wallin On Aug. 19, 2013, 11:11 a.m., Inge Wallin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/112154/ > ----------------------------------------------------------- > > (Updated Aug. 19, 2013, 11:11 a.m.) > > > Review request for KDE Edu and Frederik Gladhorn. > > > Description > ------- > > Currently Parley has a setting for editing language settings such as fontsize in the editor and practice and also other, more language specific things. But this is extremely well hidden in the editor mode only and under the Edit menu. > > This patch makes the language settings available also during practice. > > You could argue that settings for the language itself should be separated from viewing settings but this is not the topic of this patch. > > > Diffs > ----- > > src/editor/editor.cpp 6511a69 > src/practice/practicemainwindow.cpp 9f23ae9 > src/practice/practiceui.rc 96c4a06 > > Diff: http://git.reviewboard.kde.org/r/112154/diff/ > > > Testing > ------- > > > Thanks, > > Inge Wallin > > --===============0027531306581157332== Content-Type: text/html; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit
This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112154/

src/editor/editor.cpp (Diff revision 1)
void EditorWindow::initActions()
364
                          SLOT(languageProperties()), actionCollection());
Hmm, seems like an extra newline, yes.  I'll check it.

- Inge


On August 19th, 2013, 11:11 a.m. UTC, Inge Wallin wrote:

Review request for KDE Edu and Frederik Gladhorn.
By Inge Wallin.

Updated Aug. 19, 2013, 11:11 a.m.

Description

Currently Parley has a setting for editing language settings such as fontsize in the editor and practice and also other, more language specific things.  But this is extremely well hidden in the editor mode only and under the Edit menu.

This patch makes the language settings available also during practice.

You could argue that settings for the language itself should be separated from viewing settings but this is not the topic of this patch.

Diffs

  • src/editor/editor.cpp (6511a69)
  • src/practice/practicemainwindow.cpp (9f23ae9)
  • src/practice/practiceui.rc (96c4a06)

View Diff

--===============0027531306581157332==-- --===============5761409103842317774== 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 --===============5761409103842317774==--