----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109021/#review32675 -----------------------------------------------------------
core/textdocumentgenerator.cpp <http://git.reviewboard.kde.org/r/109021/#comment24305> Aren't we still leaking new TextDocumentSettingsWidget()? core/textdocumentsettings.cpp <http://git.reviewboard.kde.org/r/109021/#comment24307> I'm missing some kind of label, the combobox looks a bit lonely out there. core/textdocumentsettings.cpp <http://git.reviewboard.kde.org/r/109021/#comment24306> Have you tried a KFontRequester instead of a KFontComboBox? (So you can choose size, etc)? core/textdocumentsettings.cpp <http://git.reviewboard.kde.org/r/109021/#comment24308> Maybe add a label to this so we can call addRow ( const QString & labelText, QWidget * field )? - Albert Astals Cid On May 16, 2013, 10:02 p.m., Azat Khuzhin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/109021/ > ----------------------------------------------------------- > > (Updated May 16, 2013, 10:02 p.m.) > > > Review request for Okular, Albert Astals Cid and Eike Hein. > > > Description > ------- > > Development history: > https://github.com/azat/okular/compare/master...font-selector-for-plain-text-formats > > http://quickgit.kde.org/?p=clones%2Fokular%2Fazatkhuzhin%2Fokular.git&a=commitdiff&h=font-selector-for-plain-text-formats&hp=master > (but it has some delay, about a day or so while this clone will be updated, > maybe because of mirrors?) > > Link to thread from mailing list: > http://comments.gmane.org/gmane.comp.kde.devel.okular/13279 > > > Diffs > ----- > > CMakeLists.txt 543e6df > conf/textdocumentsettings.ui PRE-CREATION > core/textdocumentgenerator.h dd75c5c > core/textdocumentgenerator.cpp f370ded > core/textdocumentgenerator_p.h 749d6f2 > core/textdocumentsettings.h PRE-CREATION > core/textdocumentsettings.cpp PRE-CREATION > core/textdocumentsettings_p.h PRE-CREATION > generators/epub/CMakeLists.txt f076ed9 > generators/epub/generator_epub.h cef2879 > generators/epub/generator_epub.cpp 59bb2bf > generators/epub/libokularGenerator_epub.desktop 5c853a3 > generators/fictionbook/generator_fb.h a898397 > generators/fictionbook/generator_fb.cpp 2317083 > generators/fictionbook/libokularGenerator_fb.desktop 099268c > generators/ooo/converter.cpp 1124e2a > generators/ooo/generator_ooo.h 3441c7a > generators/ooo/generator_ooo.cpp 793ee58 > generators/ooo/libokularGenerator_ooo.desktop 328ae26 > generators/txt/CMakeLists.txt 5a126b7 > generators/txt/generator_txt.h 5c15ec4 > generators/txt/generator_txt.cpp 93ca4aa > generators/txt/libokularGenerator_txt.desktop 235e23d > > Diff: http://git.reviewboard.kde.org/r/109021/diff/ > > > Testing > ------- > > Tested manually > > > Thanks, > > Azat Khuzhin > >
_______________________________________________ Okular-devel mailing list Okular-devel@kde.org https://mail.kde.org/mailman/listinfo/okular-devel
