> On May 8, 2013, 6:02 p.m., Albert Astals Cid wrote: > > core/textdocumentsettings.h, line 29 > > <http://git.reviewboard.kde.org/r/109021/diff/2/?file=142647#file142647line29> > > > > We need to install this class so generators outside the okular repo can > > use it. This means you need to add it to the install list somewhere in the > > CMakeLists.txt and that you should add a d-pointer so it's possible to keep > > Binary Compatibility when we need to add more fields, etc. Also some > > documentation at least mentioning an @since is needed > > Azat Khuzhin wrote: > It is already in install() > > > https://git.reviewboard.kde.org/r/109021/diff/2/?file=142647#file142647line29 > > What about private class - I agree, will be done.
I think here we must add map of widgets/objects into TextDocumentSettings/TextDocumentSettingsSkeleton To allow manually adding fields from generators. Don't you think so? - Azat ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109021/#review32266 ----------------------------------------------------------- On May 11, 2013, 7:40 p.m., Azat Khuzhin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/109021/ > ----------------------------------------------------------- > > (Updated May 11, 2013, 7:40 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 > > 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
